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

fix: mlflow versioning #77

Merged
merged 3 commits into from
Nov 6, 2023
Merged

fix: mlflow versioning #77

merged 3 commits into from
Nov 6, 2023

Conversation

rcmalli
Copy link
Collaborator

@rcmalli rcmalli commented Nov 3, 2023

Summary

This PR fixes Mlflow versioning including skinny package. Previously, later versions of mlflow-skinny could be installed even if we fix mlflow package version. In the future releases, we can drop the dependency of mlflow and add only mlflow-skinny.

Type of Change

  • Bug fix (non-breaking change that solves an issue)

Checklist

  • I have tested my changes locally and they work as expected. (Please describe the tests you performed.)
  • I have added unit tests for my changes, or updated existing tests if necessary.
  • I have updated the documentation, if applicable.
  • I have installed pre-commit and run locally for my code changes.

@rcmalli rcmalli changed the title Hotfix/mlflow versioning fix: mlflow versioning Nov 3, 2023
@rcmalli rcmalli merged commit b49115f into main Nov 6, 2023
@rcmalli rcmalli deleted the hotfix/mlflow_versioning branch November 6, 2023 10:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants