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

Got 'error: duplicate parameter name' when ./generate_run test in userproject/Izh_sparse_project #416

Closed
luyi256 opened this issue Apr 22, 2021 · 4 comments
Labels
Milestone

Comments

@luyi256
Copy link

luyi256 commented Apr 22, 2021

I'm using genn-4.4.0. After entering userproject/Izh_sparse_project, use make and ./generate_run test, I got

/home/xxxx/genn/genn-4.4.0/userproject/Izh_sparse_project/model/IzhSparse_CODE/definitionsInternal.h(164): error: duplicate parameter name

1 error detected in the compilation of "/tmp/tmpxft_0002235d_00000000-6_neuronUpdate.cpp1.ii".
optimizeBlockSize: NVCC failed
genn-buildmodel.sh:88: error 50: command failure
ERROR: Following call failed with status 12800:
cd model && genn-buildmodel.sh IzhSparse.cc && make -j 56 clean all
Exiting...

My OS is Linux Ubuntu, CUDA version is 10.1. I think it may be a little bug in genn.
Thanks for your help.

@neworderofjamie neworderofjamie added this to the GeNN 4.5.0 milestone Apr 22, 2021
@neworderofjamie
Copy link
Contributor

neworderofjamie commented Apr 28, 2021

Thanks for this bug report. Sadly, the only workaround right now is not to use the built-in FixedNumberPost connector (which the IzhSparse model does) but, on the bright side, I don't think that model is used that widely elsewhere.

The underlying issue is the result of a name conflict between the FixedNumberPost connector parameter "rowLength" and the merged structure field used to store the row length. Similar class of issue to #385 and similarly the solution is to go through and put underscores in front of all internally-generated variables etc.

@luyi256
Copy link
Author

luyi256 commented May 2, 2021

@neworderofjamie Got it, thanks a lot.

@luyi256 luyi256 closed this as completed May 2, 2021
@neworderofjamie
Copy link
Contributor

This is a legitimate bug so I'd like to keep it open!

@neworderofjamie neworderofjamie removed this from the GeNN 4.5.0 milestone Jul 15, 2021
@neworderofjamie
Copy link
Contributor

This parameter has been renamed to "num"

@neworderofjamie neworderofjamie added this to the GeNN 5.0.0 milestone Mar 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants