add instructlab-training[hpu] to requirements/hpu.txt#2641
add instructlab-training[hpu] to requirements/hpu.txt#2641mergify[bot] merged 1 commit intoinstructlab:mainfrom
Conversation
JamesKunstle
force-pushed
the
requirements-hpu
branch
from
027cfd6 to
5a9219e
Compare
nathan-weinberg
left a comment
nathan-weinberg
left a comment
There was a problem hiding this comment.
Holding for instructlab/training#320 to merge
nathan-weinberg
added
the
hold
| #habana-torch-dataloader | ||
|
|
||
| # Extra dependencies for Intel Gaudi cards | ||
| instructlab-training[hpu]>=0.5.0 |
There was a problem hiding this comment.
This should actually be held until 0.6.0 is cut and exists, since 0.6.0 will be the first version to feature the hpu extra
There was a problem hiding this comment.
So the version here should actually be 0.6.0 instead of 0.5.0?
There was a problem hiding this comment.
@JamesKunstle please fix that when you get a chance, ty!
There was a problem hiding this comment.
v0.6.0 has been cut now, so can be used as min
There was a problem hiding this comment.
Got it will update asap!
JamesKunstle
force-pushed
the
requirements-hpu
branch
from
5a9219e to
2a006ff
Compare
|
@Maxusmusti @nathan-weinberg Updated the PR to require training 0.6.0 as min bound for all extras |
nathan-weinberg
removed
the
hold
in order for instructlab to be installed for hpus, require that we also install the dependencies in the hpu set of requirements for instructlab-training Signed-off-by: James Kunstle <jkunstle@redhat.com>
JamesKunstle
force-pushed
the
requirements-hpu
branch
from
2a006ff to
829482c
Compare
|
rebased to remove bumps to cuda and rocm extras from this commit |
5 participants
in order for instructlab to be installed for hpus, require that we also
install the dependencies in the hpu set of requirements for
instructlab-training
Signed-off-by: James Kunstle jkunstle@redhat.com