-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
summarize_graph show framework error when input saved_model is selected #6726
Merged
rkazants
merged 3 commits into
openvinotoolkit:master
from
vgavrilo:vgavrilo/summarize-graph
Jul 22, 2021
Merged
summarize_graph show framework error when input saved_model is selected #6726
rkazants
merged 3 commits into
openvinotoolkit:master
from
vgavrilo:vgavrilo/summarize-graph
Jul 22, 2021
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
vgavrilo
requested review from
a team,
iimironov,
rkazants,
popovaan and
lazarevevgeny
and removed request for
a team
July 21, 2021 06:57
@iimironov, @rkazants, @popovaan, @lazarevevgeny, please, see this PR (#6726) |
iimironov
reviewed
Jul 21, 2021
@iimironov, @rkazants, @popovaan, @lazarevevgeny, please, see this PR (#6726) |
popovaan
reviewed
Jul 21, 2021
@iimironov, @rkazants, @popovaan, @lazarevevgeny, please, see this PR (#6726) |
rkazants
approved these changes
Jul 22, 2021
@iimironov, @popovaan, @lazarevevgeny, please, see this PR (#6726) |
popovaan
approved these changes
Jul 22, 2021
rnugmanx
pushed a commit
to rnugmanx/openvino
that referenced
this pull request
Aug 26, 2021
andrei-cv
pushed a commit
to andrei-cv/openvino
that referenced
this pull request
Aug 30, 2021
akuporos
pushed a commit
to akuporos/openvino
that referenced
this pull request
Sep 29, 2021
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.
Root cause analysis: For some TF saved_model, we have a framework error after running the script
summarize_graph.py
.The reason of this error is the following: the value of the command line key
--saved_model_tags
must be considered as a group of tag(s) of the MetaGraphDef to load, in string format, separated by commas. Hence, this value must be split into a list of strings by commas. But such splitting isn't performed.Solution: add splitting the value of the command line key
--saved_model_tags
by commas.Tickets:
Code:
Validation:
Documentation: