Megvii-BaseDetection / YOLOX

YOLOX is a high-performance anchor-free YOLO, exceeding yolov3~v5 with MegEngine, ONNX, TensorRT, ncnn, and OpenVINO supported. Documentation: https://yolox.readthedocs.io/
Apache License 2.0
9.15k stars 2.15k forks source link

Adding mlflow logging functionality to yolox #1772

Closed Im-Himanshu closed 1 month ago

Im-Himanshu commented 1 month ago

Needed integration of yolox to log experiments with mlflow, for that created this pull request that create another option in -l --logger argument to be "mlflow".

works on environment file (.env) in the root folder of the projects.

required additional dependency of mlflow and python-dotenv failing which error is raised if logger is set to mlflow.

CLAassistant commented 1 month ago

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

:white_check_mark: Im-Himanshu
:x: jgoyalh1


jgoyalh1 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

Im-Himanshu commented 1 month ago

committed with wrong username, deleting to push in different pull request.