Skip to content

Remove optimum dependency#407

Merged
JamesKunstle merged 1 commit intoinstructlab:mainfrom
fabiendupont:hpu-remove-optimum-deps
Jan 23, 2025
Merged

Remove optimum dependency#407
JamesKunstle merged 1 commit intoinstructlab:mainfrom
fabiendupont:hpu-remove-optimum-deps

Conversation

@fabiendupont
Copy link
Contributor

Follows-up on instructlab/instructlab#2964

Signed-off-by: Fabien Dupont <fdupont@redhat.com>
@mergify mergify bot added ci-failure dependencies Pull requests that update a dependency file labels Jan 22, 2025
Copy link
Contributor

@tiran tiran left a comment

Choose a reason for hiding this comment

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

LGTM

Please merge after the InstructLab PR has landed.

@nathan-weinberg nathan-weinberg requested a review from a team January 22, 2025 17:57
@mergify mergify bot added the one-approval label Jan 23, 2025
@JamesKunstle
Copy link
Contributor

@tiran @nathan-weinberg does merging this actually need to wait for the CLI PR or can it just merge now?

@nathan-weinberg
Copy link
Member

The other PR has merged so we are good to do this either way 👍

@JamesKunstle JamesKunstle merged commit a1c700d into instructlab:main Jan 23, 2025
15 of 16 checks passed
@mergify mergify bot removed the one-approval label Jan 23, 2025
@JamesKunstle
Copy link
Contributor

@nathan-weinberg Alright rad!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-failure dependencies Pull requests that update a dependency file hold

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants