Skip to content

Fix export_models.py with predownloaded models #3836

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

Open
atobiszei wants to merge 1 commit into
base: main
Choose a base branch
from
+0 −1
Open

Fix export_models.py with predownloaded models #3836

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 0 additions & 1 deletion demos/common/export_models/export_model.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -386,7 +386,6 @@ def export_text_generation_model(model_repository_path, source_model, model_name
### Export model
if os.path.isfile(os.path.join(source_model, 'openvino_model.xml')) or os.path.isfile(os.path.join(source_model, 'openvino_language_model.xml')):
print("OV model is source folder. Skipping conversion.")
model_path = source_model
Copy link
Collaborator

@dtrawins dtrawins Dec 5, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My recollection is that this condition was for the scenario of source model from NFS storage with RO access. That was to be absolute path.

Copy link
Collaborator Author

@atobiszei atobiszei Jan 26, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Right now when used for 1st time script pull the model:
python ~/git_repos/ovms1/demos/common/export_models/export_model.py text_generation --source_model Qwen/Qwen3-8B --weight-format int8 --model_repository_path ./ --tool_parser hermes3 --overwrite_models --enable_prefix_caching
Then in ./config.json we have "base_path": "Qwen/Qwen3-8B", and in graph.pbtxt : models_path: "./",. After second launch in graph.pbtxt we have: models_path: "Qwen/Qwen3-8B",, which fails since it would require graph.pbtxt to be in: ````./Qwen/Qwen3-8B/Qwen/Qwen3-8B/``` directory.

Additionally when launched with slightly different command (just going 1 step above in directory tree):

python ~/git_repos/ovms1/demos/common/export_models/export_model.py text_generation --source_model Qwen/Qwen3-8B --weight-format int8 --model_repository_path ./models3  --tool_parser hermes3 --overwrite_models --enable_prefix_caching

This works fine (paths in graph.pbtxt are fine).
Either way we would need a way to tell when to create graph.pbtxt in separate directory tree (with RW).

elif source_model.startswith("OpenVINO/"):
if precision:
print("Precision change is not supported for OpenVINO models. Parameter --weight-format {} will be ignored.".format(precision))
Expand Down