Fix default_logdir import for transformers v5

[elias-aouad]

Fix an import for better compatibility with transformers v5

[peter-clark-moj]

would be great if someone could jump on this. given transformers versions <5.3.0 are considered to have a high danger security vulnerability.

[bigabig]

Please merge this, it would be nice to be able to work with Transformers v5!

[elias-aouad]

@tomaarsen I am pinging you in case you're still a maintainer of this repo. This is a one-liner PR, would it be possible to merge it ? It would help a lot !
Cheers 🍻
PS: Amazing work you did with this repo, it helps a lot :)

[tomaarsen]

Apologies for the delay. I think it might be safer if we used a try-except ImportError as I imagine your new proposal won't work with older transformers versions.

• Tom Aarsen

[elias-aouad]

Apologies for the delay. I think it might be safer if we used a try-except ImportError as I imagine your new proposal won't work with older transformers versions.

• Tom Aarsen

Thank you for the reply, it's all done now :)

[elias-aouad]

@tomaarsen sorry for the second ping, the PR is ready now with the try-except logic. It would be of great help if we could merge it 🙏

Thanks a lot !