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

[CSTP] Possible STP time delay bug #20

Open
nmsutton opened this issue Dec 9, 2022 · 0 comments
Open

[CSTP] Possible STP time delay bug #20

nmsutton opened this issue Dec 9, 2022 · 0 comments

Comments

@nmsutton
Copy link
Contributor

nmsutton commented Dec 9, 2022

In snn_gpu_module.cu, the code comment "FIXME should be adjusted for delay" for the line setting "ind_minus" value indicates that synaptic delays greater than 1ms have not been implemented in STP calculations. It appears this may affect all STP processing (connection-specific and non-specific). While synaptic delays may be processed in other code calculations, at least in STP calculations they may be missing. 1ms delays may work correctly with the code. This possible issue should be investigated and perhaps a note can be made about this not yet working in the user guide/docs for STP/connect() function in CARLsim.

In the main branch CARLsim code's snn_manager.cpp (verifyNetwork()) there is a check for STP being enabled and delays > 1 with the error "STP with delays > 1 ms is currently not supported." In the feat/ca3net CARLsim branch, when JK_CA3_SNN is enabled (code for CA3 net) this error is disabled. Therefore the ability to have STP with delays greater than 1 ms may only be possible in the feat/ca3net branch code. In that branch's code possibly delays >1 with STP are not processed correctly.

@nmsutton nmsutton changed the title Possible STP time delay bug [CSTP] Possible STP time delay bug Dec 24, 2022
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