[GPU][TESTS] Fix weightless caching tests #27163
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
build_jenkins |
e-ddykim
approved these changes
Oct 22, 2024
p-durandin
approved these changes
Oct 22, 2024
|
to be rebased after #27171 merge |
Merged
via the queue into
openvinotoolkit:master
with commit Oct 24, 2024
17f1601
143 of 145 checks passed
CuriousPanCake
pushed a commit
to CuriousPanCake/openvino
that referenced
this pull request
Nov 6, 2024
Currently the tests have weightless caching enabled, but the models tested contain no constants eligible for weightless caching. This fixes the situation. To elaborate - all the models have f32 dtype constants by default. In these particular models all of the constants which would otherwise be eligible for weightless caching undergo some precision transformation which invalidates this eligibilty. The PR changes the dtype of the constants to prevent that and deletes the models which contain no constants eligible for weightless caching even after the dtype change.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Currently the tests have weightless caching enabled, but the models tested contain no constants eligible for weightless caching. This fixes the situation.
To elaborate - all the models have f32 dtype constants by default. In these particular models all of the constants which would otherwise be eligible for weightless caching undergo some precision transformation which invalidates this eligibilty. The PR changes the dtype of the constants to prevent that and deletes the models which contain no constants eligible for weightless caching even after the dtype change.