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

Changes in approximate convolution #16

Open
ayeshamkzhh opened this issue May 2, 2021 · 0 comments
Open

Changes in approximate convolution #16

ayeshamkzhh opened this issue May 2, 2021 · 0 comments

Comments

@ayeshamkzhh
Copy link

Hi, I was wondering if approx_nn_conv_ops_ref.cpp needs to be edited and then, the libApproxGPUOpsTF.so need to be created again if we want to make any changes in the convolution. It seems like in line 300 of this file, prodValueQ is meant for storing the approximate multiplication result in convolution.

I tried to make some changes in convolution there and run the cmake and make commands again but its quite strange that no changes affect the accuracy.

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

1 participant