Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Comments

Close side panel

Follow up to safetensor support#1035

Merged
mergify[bot] merged 1 commit intoinstructlab:maininstructlab/instructlab:mainfrom
cdoern:model-agnostic-downloadcdoern/instructlab:model-agnostic-downloadCopy head branch name to clipboard
Apr 29, 2024
Merged

Follow up to safetensor support#1035
mergify[bot] merged 1 commit intoinstructlab:maininstructlab/instructlab:mainfrom
cdoern:model-agnostic-downloadcdoern/instructlab:model-agnostic-downloadCopy head branch name to clipboard

Conversation

@cdoern
Copy link
Contributor

@cdoern cdoern commented Apr 29, 2024

add --hf-token that defaults to the envvar HF_TOKEN, use snapshot_download to get the whole repo, and switch the check for HF_TOKEN to only see if instructlab is in the repo name, not full merl url

@cdoern cdoern force-pushed the model-agnostic-download branch 2 times, most recently from 242cb4e to 8f42170 Compare April 29, 2024 19:57
@alimaredia alimaredia added the hold In-progress PR. Tag should be removed before merge. label Apr 29, 2024
@alimaredia alimaredia self-requested a review April 29, 2024 20:27
@alimaredia alimaredia requested a review from russellb April 29, 2024 20:28
add `--hf-token` that defaults to the envvar HF_TOKEN, use snapshot_download to get the whole repo, and switch the check for HF_TOKEN to only see if instructlab is in the repo name, not full merl url

Signed-off-by: Charlie Doern <cdoern@redhat.com>
@cdoern cdoern force-pushed the model-agnostic-download branch from 8f42170 to b26d90a Compare April 29, 2024 20:33
@alimaredia alimaredia removed the hold In-progress PR. Tag should be removed before merge. label Apr 29, 2024
@mergify mergify bot merged commit 3e7a7a1 into instructlab:main Apr 29, 2024
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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.