search

deepset-ai / FARM

:house_with_garden: Fast & easy transfer learning for NLP. Harvesting language models for the industry. Focus on Question Answering.
https://farm.deepset.ai
Apache License 2.0
1.74k stars 247 forks source link

requirements.txt -> loosen up mlflow restriction #834

Closed codingmercy1993 closed 2 years ago

codingmercy1993 commented 3 years ago

Question Hey guys! FARM has strictly specified mlflow<=1.13.1 version in requirements. This version is more than one year old. We want to use FARM in a corporate context where we need to use a relatively new mlflow version (mlflow == 1.19.0). Unfortunately this procudes conflicts that we can not resolve. We already discuss about forking FARM to manage these conlicts.

What do you think about integrating a newer mlflow version or loosen up your restrictions a bit (e.g. mlflow~= 1.13.*) in requirements.txt? I can do it on my own, check if everything works fine and then create pull request. Just want to know, maybe version was strictly specified for some particular reason.

Thanks you! :-)

Timoeller commented 3 years ago

Hey sure, this is a good idea.

Could you create a PR and test if the logging works for some of the examples in combination with our https://public-mlflow.deepset.ai/#/ server?

stale[bot] commented 2 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 21 days if no further activity occurs.