-
-
Notifications
You must be signed in to change notification settings - Fork 8.9k
Support embedding models in V1 #16188
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
Changes from 7 commits
f36c4f9
acf4638
b13bbc0
8debea0
8d97b9c
d60b22b
6bebbb8
6dafd71
e2724a2
56ff6cd
fc57edd
64a0e62
4014d41
87a95a8
902c129
2c68855
8afd8f5
7762976
d7537ae
a9e7747
17520bd
90c611a
dec2441
a5e83f4
187f69b
69a0332
a9f1721
4b066a3
43a26dc
ca34513
bf3033d
67bf727
93b6361
d916b88
bad4211
35d9bd9
dcc6100
a4f85b5
a5f328a
7c5be88
29b75c9
6aa204c
e81470c
20e7140
6bc1e3d
22825bd
c889b2e
24462e4
b5f21f2
79d1b95
b3a0491
b4ab556
1a82e56
a66801b
660dd9c
808c996
cdd70c9
0832115
10bbf74
ee892aa
2e12eba
14fcf24
0624435
706fdb2
051f6d4
214cf06
8193bd0
65b8377
33d7f74
4ee822a
7242731
a4f460b
35ca640
17f6177
3f0d42e
74d73cc
e6a66dc
4cca774
8ef1982
28d00d1
e634f60
053475c
6228f64
42c802a
f771a19
02c47ad
1fd252c
c5c0d97
acfc9cc
225b808
2b86c13
2983252
58c556d
878d56a
2db273f
114af27
bc0219d
221f013
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -1439,12 +1439,6 @@ def _is_v1_supported_oracle(self, model_config: ModelConfig) -> bool: | |
recommend_to_remove=False) | ||
return False | ||
|
||
# No Embedding Models so far. | ||
if model_config.task not in ["generate"]: | ||
_raise_or_fallback(feature_name=f"--task {model_config.task}", | ||
recommend_to_remove=False) | ||
return False | ||
Comment on lines
-1352
to
-1356
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Did you mean to enable all tasks here?
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'll double check the "transcription" task, but the others yes. Is this causing a problem? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Nope, it just caused a conflcit in my branch where I had enabled "transcription" and thought maybe it had enabled more than you intended. It's fine if you mean to! I'm not sure about transcription, either. I know it wouldn't work with whisper, but that'll still get blocked because the model is marked as v0-only. Since all models should have the V0-only marker where needed, this check probably isn't necessary. |
||
|
||
# No Mamba or Encoder-Decoder so far. | ||
if not model_config.is_v1_compatible: | ||
_raise_or_fallback(feature_name=model_config.architectures, | ||
|
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.
Why is it not supported?
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.
Some models were returning incorrect output after compiling. I left this as a follow-up work