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

Hotfix: add custom callback to avoid azureml 100 params limitation #86

Merged
merged 2 commits into from
Sep 9, 2022

Conversation

jfomhover
Copy link
Contributor

PR into Azure/mlops-v2

We're facing a blocker when using HF's default callback to mlflow backend, that way too many parameters are sent to mlflow (more than 100). This PR creates a custom callback to avoid this issue, while the limitation is being discussed in AzureML.

Checklist

I have:

  • read and followed the contributing guidelines

Changes

  • removes the default mlflow callback
  • implement a custom callback using only metric reporting

@jfomhover jfomhover added the 🪲bug Something isn't working label Sep 7, 2022
@jfomhover jfomhover marked this pull request as ready for review September 7, 2022 05:32
@cindyweng cindyweng merged commit 77ec671 into main Sep 9, 2022
@cindyweng cindyweng deleted the jfomhover/hotfixmlflowintegration branch September 9, 2022 13:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🪲bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants