Skip to content
This repository has been archived by the owner on Dec 20, 2024. It is now read-only.

Update logger.py #169

Merged
merged 2 commits into from
Nov 27, 2024
Merged

Update logger.py #169

merged 2 commits into from
Nov 27, 2024

Conversation

HCookie
Copy link
Member

@HCookie HCookie commented Nov 27, 2024

Remove ‘metadata.dataset.variables_metadata’ from parms to log

Remove ‘metadat.dataset.variables_metadata’ from parms to log
@HCookie HCookie added the no changelog Flag a PR as not requiring a changelog update label Nov 27, 2024
@HCookie HCookie self-assigned this Nov 27, 2024
Copy link
Member

@mchantry mchantry left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks.

@mchantry
Copy link
Member

Tested, this removes excess metadata from MLFlow servers. Great work.

@HCookie HCookie merged commit 112d78f into develop Nov 27, 2024
117 checks passed
@HCookie HCookie deleted the hotfix/mlflow-dataset-logging branch November 27, 2024 10:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
no changelog Flag a PR as not requiring a changelog update
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants