Skip to content
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

"constant" not found #102

Open
YuqiData-UW opened this issue Feb 4, 2023 · 3 comments
Open

"constant" not found #102

YuqiData-UW opened this issue Feb 4, 2023 · 3 comments

Comments

@YuqiData-UW
Copy link

According to

"attr": self.fetch_attr_to_dict(node),
"constant" should always be saved under graph[in_node]["attr"]["attr"], but we got
graph[in_node]["attr"]["constant"]. Is this a bug or I missed an edge case?

@YuqiData-UW YuqiData-UW changed the title Is this a bug? "constant" not found Feb 4, 2023
@JiahangXu
Copy link
Collaborator

Could you please add a test case with bug information? I think it would help us to locate the problem.

@JiahangXu
Copy link
Collaborator

Hi Yuqi, I noticed that you have reported a number of issues with the frozenpb converter. I would like to understand more about your project and if you have any test cases that can help demonstrate the problems you are encountering. It is possible that there may be some edge cases that have not been considered previously.

@YuqiData-UW
Copy link
Author

Hi Jiahang,

Thank you for following up. I have found the nn-meter project to be very impressive. Currently, I am testing it by converting TF2 and Torch models to frozenpb and evaluating its performance. I have brought a few issues which I believe are not unique to my personal project, but may affect other use cases as well. There are also some issues about calculations I am still trying to understand.

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

No branches or pull requests

2 participants