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

MLFlow logger bug #16443

Closed
riklopfer opened this issue Jan 19, 2023 · 1 comment
Closed

MLFlow logger bug #16443

riklopfer opened this issue Jan 19, 2023 · 1 comment
Labels
needs triage Waiting to be triaged by maintainers

Comments

@riklopfer
Copy link

Bug description

Wrong variable used

https://github.com/Lightning-AI/lightning/blob/8f736372ed27a9d3211febc4b8bbdd337b91dad1/src/pytorch_lightning/loggers/mlflow.py#L250

How to reproduce the bug

No response

Error messages and logs

  File "/usr/local/Caskroom/miniforge/base/envs/cf-gloss-wsd/lib/python3.9/site-packages/pytorch_lightning/trainer/trainer.py", line 1159, in _log_hyperparams
    logger.log_hyperparams(hparams_initial)
  File "/usr/local/Caskroom/miniforge/base/envs/cf-gloss-wsd/lib/python3.9/site-packages/lightning_utilities/core/rank_zero.py", line 24, in wrapped_fn
    return fn(*args, **kwargs)
  File "/usr/local/Caskroom/miniforge/base/envs/cf-gloss-wsd/lib/python3.9/site-packages/pytorch_lightning/loggers/mlflow.py", line 252, in log_hyperparams
    self.experiment.log_batch(run_id=self.run_id, params=params_list)
  File "/usr/local/Caskroom/miniforge/base/envs/cf-gloss-wsd/lib/python3.9/site-packages/mlflow/tracking/client.py", line 965, in log_batch
    self._tracking_client.log_batch(run_id, metrics, params, tags)
  File "/usr/local/Caskroom/miniforge/base/envs/cf-gloss-wsd/lib/python3.9/site-packages/mlflow/tracking/_tracking_service/client.py", line 389, in log_batch
    self.store.log_batch(
  File "/usr/local/Caskroom/miniforge/base/envs/cf-gloss-wsd/lib/python3.9/site-packages/mlflow/store/tracking/file_store.py", line 1014, in log_batch
    _validate_batch_log_data(metrics, params, tags)
  File "/usr/local/Caskroom/miniforge/base/envs/cf-gloss-wsd/lib/python3.9/site-packages/mlflow/utils/validation.py", line 303, in _validate_batch_log_data
    _validate_param(param.key, param.value)
  File "/usr/local/Caskroom/miniforge/base/envs/cf-gloss-wsd/lib/python3.9/site-packages/mlflow/utils/validation.py", line 150, in _validate_param
    _validate_param_name(key)
  File "/usr/local/Caskroom/miniforge/base/envs/cf-gloss-wsd/lib/python3.9/site-packages/mlflow/utils/validation.py", line 217, in _validate_param_name
    if not _VALID_PARAM_AND_METRIC_NAMES.match(name):
TypeError: expected string or bytes-like object

Environment

Current environment
#- Lightning Component (e.g. Trainer, LightningModule, LightningApp, LightningWork, LightningFlow):
#- PyTorch Lightning Version (e.g., 1.5.0):
#- Lightning App Version (e.g., 0.5.2):
#- PyTorch Version (e.g., 1.10):
#- Python version (e.g., 3.9):
#- OS (e.g., Linux):
#- CUDA/cuDNN version:
#- GPU models and configuration:
#- How you installed Lightning(`conda`, `pip`, source):
#- Running environment of LightningApp (e.g. local, cloud):

More info

No response

@riklopfer riklopfer added the needs triage Waiting to be triaged by maintainers label Jan 19, 2023
@riklopfer riklopfer mentioned this issue Jan 19, 2023
12 tasks
@riklopfer
Copy link
Author

duplicates #16411

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs triage Waiting to be triaged by maintainers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant