Skip to content
This repository has been archived by the owner on May 9, 2024. It is now read-only.

Binary support for ebd-all-minilm #98

Closed
wants to merge 21 commits into from
Closed

Binary support for ebd-all-minilm #98

wants to merge 21 commits into from

Conversation

nsosio
Copy link
Contributor

@nsosio nsosio commented Nov 3, 2023

Fixes #97

biswaroop1547 and others added 17 commits October 26, 2023 11:50
@biswaroop1547
add: manifest file for stablebeluga2 of type:process using petals
401d68e
@biswaroop1547
update: changes to manifest from review
a710295
@biswaroop1547
revert: manifest configurability - from review comments
90df6d5
@biswaroop1547
update: servecommand with model-id
4ecb955
@biswaroop1547
fix: port in readme
246ffe5
@biswaroop1547
fix: binary links path
48823d5
@tiero
Update manifest.json
3deb536
@tiero
Update manifest.json
ad04af8
@actions-user
Update manifests.json
ef52d6b
@biswaroop1547
del: stable-beluga docker manifest and files, kept only bash script t…
a366fe8 
…ype process manifest, minor edits
@tiero
fix: whisper tiny
ae7c58b
@actions-user
Update manifests.json
23bd284
@biswaroop1547
update: readme responses changes, description on manifest
6f469fb
@biswaroop1547
update: only universal binary url is required for manifest
c4aaea1
@casperdcl
Merge pull request bit-gpt#88 from biswaroop1547/stable-beluga-process
35477f0 
manifest file for stablebeluga2 of type: `process` using petals
@actions-user
Update manifests.json
307ad63
@nsosio
added binary ebd-all-minilm
ac421ab
@nsosio
Copy link
Contributor Author

nsosio commented Nov 3, 2023

\cc @casperdcl @tiero

@nsosio nsosio changed the base branch from dev to v1 November 3, 2023 12:13
@casperdcl
Copy link
Contributor

casperdcl commented Nov 3, 2023 •
edited
Loading

casperdcl
casperdcl suggested changes Nov 3, 2023
View reviewed changes
embeddings-all-minilm-l6-v2-binary/manifest.json Outdated Show resolved Hide resolved
embeddings-all-minilm-l6-v2-binary/manifest.json Outdated Show resolved Hide resolved
embeddings-all-minilm-l6-v2-binary/manifest.json Outdated Show resolved Hide resolved
@nsosio nsosio marked this pull request as ready for review November 3, 2023 13:27
@casperdcl
Copy link
Contributor

this PR probably needs rebasing :)

@nsosio
Copy link
Contributor Author

nsosio commented Nov 3, 2023

What should be the path used in the model path where the files from HF are downloaded? The service expects --model-path to be the path to the folder containing all the files

@casperdcl
Copy link
Contributor

casperdcl commented Nov 3, 2023 •
edited
Loading

--model-path . (i.e. curdir) I think (based on https://github.com/premAI-io/prem-app/blob/main/src-tauri/src/controller_binaries.rs)

@nsosio
Copy link
Contributor Author

nsosio commented Nov 4, 2023

@casperdcl added --model-dir .

@casperdcl
Copy link
Contributor

merged & released updated binary in services :)

@tiero tiero closed this Nov 7, 2023
@nsosio nsosio deleted the feat/binary-ebd-all-minilm branch December 3, 2023 09:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@casperdcl casperdcl casperdcl approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add sentence transformer binary
5 participants
@nsosio @casperdcl @tiero @biswaroop1547 @actions-user