-
Notifications
You must be signed in to change notification settings - Fork 359
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Feature] Add vis backend for MLflow. #878
Conversation
BTW, we should also update |
Co-authored-by: Mashiro <57566630+HAOCHENYE@users.noreply.github.com>
Co-authored-by: Zaida Zhou <58739961+zhouzaida@users.noreply.github.com>
Co-authored-by: Zaida Zhou <58739961+zhouzaida@users.noreply.github.com>
Co-authored-by: Zaida Zhou <58739961+zhouzaida@users.noreply.github.com>
…m/mmengine into feature-support-mlflow
Is there any way to verify that this code works with Multi GPU training? |
Thanks for your contribution. I've tested it in ddp training and it worked as expected. We'll merge this PR ASAP! |
@HAOCHENYE Thanks for your contribution. This PR will be merged after passing the CI. |
Hi @sh0622-kim !We are grateful for your efforts in helping improve this open-source project during your personal time. Welcome to join OpenMMLab Special Interest Group (SIG) private channel on Discord, where you can share your experiences, ideas, and build connections with like-minded peers. To join the SIG channel, simply message moderator— OpenMMLab on Discord or briefly share your open-source contributions in the #introductions channel and we will assist you. Look forward to seeing you there! Join us :https://discord.gg/UjgXkPWNqA Thank you again for your contribution❤ |
Nice, now we only need the ability to log custom artifacts too. |
Thanks for your contribution and we appreciate it a lot. The following instructions would make your pull request more healthy and more easily get feedback. If you do not understand some items, don't worry, just make the pull request and seek help from maintainers.
Motivation
I was reading the API documentation to use MLflow. With a recent update, I found out that MLflow is not available
I read that issue #731 is looking for a developer to develop the features I want. I hope this helps.
Modification
Add vis backend for MLflow.
BC-breaking (Optional)
While adding VIS and verifying it, a NoneType error occurred in
mmengine/model/base_module.py
Line 170: handler.stream.write
, so a conditional statement was added. Please review if there is any problem.Use cases (Optional)
Checklist