Skip to content

Conversation

@superdosh
Copy link
Contributor

  • Upgrade mlflow (needed due to a modelbench pyarrow upgrade)
  • Add scripts to validate versions across mlflow dockerfile and poetry env match

@github-actions
Copy link

github-actions bot commented Dec 9, 2025

MLCommons CLA bot All contributors have signed the MLCommons CLA ✍️ ✅

@superdosh
Copy link
Contributor Author

New scripts pass current version. Now will upgrade modelbench / mlflow.

@superdosh superdosh marked this pull request as ready for review December 9, 2025 20:30
@superdosh superdosh requested a review from a team as a code owner December 9, 2025 20:30
@superdosh superdosh requested a review from bkorycki December 9, 2025 20:30
Copy link
Contributor

@bkorycki bkorycki left a comment

Choose a reason for hiding this comment

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

lgtm!

- name: Install poetry
run: pipx install "poetry == 1.8.5"
run: pipx install poetry
Copy link
Contributor

Choose a reason for hiding this comment

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

why not specify the version anymore?

Copy link
Contributor Author

@superdosh superdosh Dec 9, 2025

Choose a reason for hiding this comment

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

We were pinned to a very old version because of some weirdness (in modelbench) that I never fully understood, but I figure that's not necessary here anymore -- I think it was actually driven by modelbench-private? Potentially pinning to some newer version is good, but at least we shouldn't be stuck on 1 if not necessary. Going to leave it open for now!

@superdosh superdosh merged commit 456712b into main Dec 9, 2025
3 checks passed
@superdosh superdosh deleted the bitrot branch December 9, 2025 22:05
@github-actions github-actions bot locked and limited conversation to collaborators Dec 9, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants