Skip to content
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

MlflowLogger fail when logging long parameters #5892

Closed
ducthienbui97 opened this issue Feb 10, 2021 · 1 comment · Fixed by #5893
Closed

MlflowLogger fail when logging long parameters #5892

ducthienbui97 opened this issue Feb 10, 2021 · 1 comment · Fixed by #5893
Labels
bug Something isn't working help wanted Open to be worked on priority: 2 Low priority task

Comments

@ducthienbui97
Copy link
Contributor

ducthienbui97 commented Feb 10, 2021

🐛 Bug

Please reproduce using the BoringModel

To Reproduce

Log anything parameters longer than 250 characters

Expected behavior

MlflowLogger not sending parameters longer than 250 characters to mlflow and log warning to user

Environment

  • PyTorch Version (e.g., 1.0):
  • OS (e.g., Linux):
  • How you installed PyTorch (conda, pip, source): pip
  • Build command you used (if compiling from source):
  • Python version: 3.7
  • CUDA/cuDNN version:
  • GPU models and configuration:
  • Any other relevant information:

Additional context

Mlflow only allow paramters to be at most 500 bytes (250 unicode characters), their limit in database is 250 characters:
https://www.mlflow.org/docs/latest/rest-api.html#log-param
mlflow/mlflow#1976
mlflow/mlflow#3931

@ducthienbui97 ducthienbui97 added bug Something isn't working help wanted Open to be worked on labels Feb 10, 2021
@tchaton tchaton added the priority: 2 Low priority task label Feb 15, 2021
@tchaton
Copy link
Contributor

tchaton commented Feb 15, 2021

Dear @ducthienbui97,

Thanks for opening a PR.

Best,
T.C

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Open to be worked on priority: 2 Low priority task
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants