Nice!
One small remark, the contributions from open-source are not mentioned in the release notes 😇
FYI @<1523701087100473344:profile|SuccessfulKoala55> (or I might be doing something wrong), but it seems the python migration code comes with carriage returns, so it fails on linux by default (one has to tr -d '\r' to use it)
EDIT: And also it defaults to /opt/allegro/data rather than the recommended /opt/clearml/data which is suggested when installing the server 🤔
It was really easy with the attached code, really 👍
I would only maybe suggest adding in the documentation, that if one uses the default recommended install location, then the script can be run without any command line arguments.
I had to momentarily look at the code to see the default paths match my own (though I could've also looked at --help default values 😛 )
Oh, you're right, I'll add that! We do reference all PRs, but you're absolutely right 🙏
@<1523707520978259968:profile|CostlyFox64> updated - my apologies for forgetting, and thanks for much for helping us improve ClearML! Please don't hesitate to call me out on it next time I miss something so important... and thanks again :)
Please let us know about the process - it's a little complicated and we hope the instructions are clear enough
@<1523701083040387072:profile|UnevenDolphin73> can you check now? I've updated the file
Perfect now 👌 (also nice cleanup of default_new_data_root duplicate code :D)
@<1523701083040387072:profile|UnevenDolphin73> thanks for noticing, I'll take a look