Skip to content

Conversation

@anhappdev
Copy link
Collaborator

@anhappdev anhappdev requested a review from a team as a code owner October 27, 2025 14:02
@anhappdev anhappdev marked this pull request as draft October 27, 2025 14:02
@github-actions
Copy link

github-actions bot commented Oct 27, 2025

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

@farook-edev
Copy link
Contributor

This probably conflicts with #1040 to some extent. Whichever is merged first will require merging master into the other and some adjustments (and resolving some merge conflicts) will be required.

Just an FYI.

@anhappdev
Copy link
Collaborator Author

This probably conflicts with #1040 to some extent. Whichever is merged first will require merging master into the other and some adjustments (and resolving some merge conflicts) will be required.

Just an FYI.

Migrating to Bazel 8 is not a high priority task compared to the LLM implementation, so I can wait until #1040 is merged and rebase this PR on it.

@farook-edev
Copy link
Contributor

farook-edev commented Nov 3, 2025

@anhappdev There's a bunch of migration work I did in #1040 to get TF 2.18.0 to work, I suggest waiting until that one's merged since it'll resolve quite a few issues when upgrading to 2.20.0 and save you some work.

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