Skip to content

Comments

Update mlflow#163

Merged
calderonjesus merged 2 commits intomainfrom
update_mlflow
Jan 20, 2026
Merged

Update mlflow#163
calderonjesus merged 2 commits intomainfrom
update_mlflow

Conversation

@calderonjesus
Copy link
Collaborator

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)

  • A breaking change in MLFlow version, from v.2 to v.3, required updating a few files.
  • The package hyperopt was not included in the uv file.
  • Currently, MLFlow does not fully support uv for uploading models. We use local storage instead of MLFlow.

What did you learn from the changes you have made?

NA

Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?

NA

Were there any challenges? If so, what issue(s) did you face? How did you overcome it?

NA

How were these changes tested?

NA

A reference to a related issue in your repository (if applicable)

NA

Checklist

  • [ X] I can confirm that my changes are working as intended

- A major version change requires refactoring
- Currently, uv gets in the way of recording the environment. Disabling log_model.
@github-actions
Copy link

Hello, thank you for your contribution. If you are a participant, please close this pull request and open it in your own forked repository instead of here. Please read the instructions on your onboarding Assignment Submission Guide more carefully. If you are not a participant, please give us up to 72 hours to review your PR. Alternatively, you can reach out to us directly to expedite the review process.

@calderonjesus calderonjesus merged commit 031d543 into main Jan 20, 2026
1 check passed
@calderonjesus calderonjesus deleted the update_mlflow branch January 20, 2026 21:13
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.

1 participant