-
Notifications
You must be signed in to change notification settings - Fork 198
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
Replace export_MiniCPM-V-2_6.py #957
Replace export_MiniCPM-V-2_6.py #957
Conversation
@@ -703,12 +703,13 @@ jobs: | |||
source ./ov/setupvars.sh | |||
python -m pip install ./thirdparty/openvino_tokenizers/[transformers] --pre --extra-index-url https://storage.openvinotoolkit.org/simple/wheels/nightly | |||
python -m pip install --upgrade-strategy eager -r ./samples/requirements.txt --pre --extra-index-url https://storage.openvinotoolkit.org/simple/wheels/nightly | |||
python ./samples/cpp/visual_language_chat/export_MiniCPM-V-2_6.py ./miniCPM-V-2_6/ | |||
python -m pip install -U "optimum<1.23" --no-dependencies |
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.
should it be moved to samples/requirements.txt
?
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.
It's a conflicting thing. The only possible thing is to add it to README.md. But I want to see if it fixes by itself before the release
A copy of #957 but installs optimum-intel from branch only for VLM
should be closed? |
#988 is merged instead |
A copy of openvinotoolkit/openvino.genai#957 but installs optimum-intel from branch only for VLM
Merge after #951