Skip to content
This repository has been archived by the owner on Jun 4, 2024. It is now read-only.

modify example with analytics-zoo #6

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

modify example with analytics-zoo #6

wants to merge 3 commits into from

Conversation

Bob-Chou
Copy link
Collaborator

modified analytics-zoo example of tf image recognition

Copy link
Contributor

@hkvision hkvision left a comment

Choose a reason for hiding this comment

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

LGTM

_ = tf.import_graph_def(graph_def, name='')
_ = tf.import_graph_def(graph_def,
input_map={'DecodeJpeg:0': image_tensor[0]},
name='zoo')
Copy link
Contributor

Choose a reason for hiding this comment

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

to make the change minimal, just use name='' in the original code instead of name='zoo'

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Thanks, I've modified the code.

"""Creates a graph from saved GraphDef file and returns a saver."""
# Creates graph from saved graph_def.pb.
w, h, c = image_array.shape
Copy link
Contributor

Choose a reason for hiding this comment

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

is the order correct? shouldn't it be HWC?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Thanks, it should be HWC. I've fixed it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants