Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Support sentence-transformers #1

Open
wants to merge 33 commits into
base: main
Choose a base branch
from
Open

Support sentence-transformers #1

wants to merge 33 commits into from

Conversation

aleksandr-mokrov
Copy link
Owner

Support sentence-transformers

aleksandr-mokrov and others added 30 commits August 8, 2024 16:14
@aleksandr-mokrov
Support sentence transformer
94ced94
@aleksandr-mokrov
FIx error
9610e0b
@aleksandr-mokrov
Separate class for SentenceTransfromers and test
12c751e
@aleksandr-mokrov
Separate class interface for Sentence Transformers
b4f4e82
@aleksandr-mokrov
Check if object has encode method
ebf0889
@aleksandr-mokrov
Formatting and change uotput checking
f3c03ec
@aleksandr-mokrov
Tokenizer init moved to __init__, sentence-tranformer pipeline in OVM…
2ee9be4 
…odelForFeatureExtraction was changed
@aleksandr-mokrov
Remove model_max_length default value
ef0ea33
@IlyasMoutawwakil
Merge branch 'main' into sentence-transformers
f4edf85
@aleksandr-mokrov @echarlaix
Update tests/openvino/test_modeling_sentence_transformers.py
531db36 
Co-authored-by: Ella Charlaix <[email protected]>
@aleksandr-mokrov @echarlaix
Update tests/openvino/test_modeling.py
876c166 
Co-authored-by: Ella Charlaix <[email protected]>
@aleksandr-mokrov
Move tokenizer initialization, other improvements
50a72ea
@aleksandr-mokrov @echarlaix
Update optimum/intel/openvino/modeling_sentence_transformers.py
d7abc3d 
Co-authored-by: Ella Charlaix <[email protected]>
@aleksandr-mokrov
Renaming OVModelForSentenceTransformer to OVSentenceTransformer
932dc75
@nikita-savelyevv
Deprecate export parameters (huggingface#886)
40194a0
@aleksandr-mokrov
Make style
6b20949
@echarlaix
Infer if the model needs to be exported (huggingface#825)
3586b5b 
* Infer if the model needs to be exported

* add test

* fix test revision

* remove infer export in pipeline

* apply comments

* fix offline mode
@aleksandr-mokrov
Move checking to init
abbdc7c
@aleksandr-mokrov @echarlaix
Update tests/openvino/test_modeling.py
89e2b07 
Co-authored-by: Ella Charlaix <[email protected]>
@aleksandr-mokrov
Add dummy_openvino_and_sentence_transformers_objects.py
40d5e4d
@aleksandr-mokrov
Merge branch 'sentence-transformers' of https://github.com/aleksandr-…
c971bff 
…mokrov/optimum-intel into sentence-transformers
@aleksandr-mokrov
refactoring
a684307
@aleksandr-mokrov @echarlaix
Update optimum/intel/utils/dummy_openvino_and_sentence_transformers_o…
1d93dee 
…bjects.py

Co-authored-by: Ella Charlaix <[email protected]>
@aleksandr-mokrov
Merge branch 'sentence-transformers' of https://github.com/aleksandr-…
ba90de1 
…mokrov/optimum-intel into sentence-transformers
@aleksandr-mokrov
Add tests to check saving and loading model
db6e2e4
@aleksandr-mokrov
Add tests to check saving and loading model
4083e7d
@aleksandr-mokrov
Support cls._library_name
2077b6c
@aleksandr-mokrov
Fix test
cebade8
@aleksandr-mokrov
Refactoring
c9b84f9
@aleksandr-mokrov
Refactoring fix
d395a65
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@aleksandr-mokrov @IlyasMoutawwakil @nikita-savelyevv @echarlaix