optionally enable export if not exported model provided
What does this PR do?
Fixes # (issue)
Before submitting
- [ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
- [ ] Did you make sure to update the documentation with your changes?
- [ ] Did you write any new necessary tests?
The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.
@echarlaix @AlexKoff88 @helena-intel could you please review and merge before next optimum release?
I don't have strong objections but I wonder if there is a more elegant way to do the same without introducing so much code that mostly duplicated Transformers logic. @echarlaix should know it better.
Also a test is required.
I don't have strong objections but I wonder if there is a more elegant way to do the same without introducing so much code that mostly duplicated Transformers logic. @echarlaix should know it better.
Also a test is required.
Yes I think we can simplify this by using find_files_matching_pattern like :
pattern = r"(.*)?openvino(.*)?\_model.xml"
ov_files = find_files_matching_pattern(
model_name_or_path,
pattern,
subfolder=subfolder,
use_auth_token=token,
revision=revision,
)
export = len(ov_files) == 0
also used in https://github.com/huggingface/optimum-intel/pull/740 for the openvino pipelines