-
Notifications
You must be signed in to change notification settings - Fork 53
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Switch To InstructLab Dolomite Repo #55
Conversation
b9bd82f
to
5d57574
Compare
Signed-off-by: Yu Chin Fabian Lim <[email protected]>
Signed-off-by: Yu Chin Fabian Lim <[email protected]>
db51c1c
to
7ea3a72
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM whenever you are ready to merge and everything works
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM since lift-and-shift
Signed-off-by: Yu Chin Fabian Lim <[email protected]>
Signed-off-by: Yu Chin Fabian Lim <[email protected]>
7ea3a72
to
5c5fc82
Compare
Signed-off-by: Yu Chin Fabian Lim <[email protected]>
@RobotSail @Maxusmusti we have published the dolomite PYPI, and i have switched to requirements file to that. Care to do the honours of merging? I believe then we can release the training package. |
This PR addresses #49
This should be merged only until instructlab/GPTDolomite#1 is merged.
instructlab-dolomite
has been published here https://pypi.org/project/instructlab-dolomite/.NOTE: the branch in the
requirements.txt
needs to be switched.Our tests show the extracted repo matches the original.
Updated picture
tested again after instructlab/GPTDolomite#3
mayank
.