search

neuronsimulator / nrn

NEURON Simulator
http://nrn.readthedocs.io
Other
411 stars 118 forks source link

Integration test for stressing inter-thread event transfer #1527

Open pramodk opened 3 years ago

pramodk commented 3 years ago

As discussed in https://github.com/BlueBrain/CoreNeuron/pull/685#issuecomment-964279999, we need a test that we can use for validating inter-thread event transfer. For both purposes - better code coverage and also identify possible bugs from refactoring work like BlueBrain/CoreNeuron/pull/685.

Michael suggested adaptation of https://github.com/nrnhines/tqperf with 1) introduction of threads 2) in case of missing event, weight should be updated so that results can be different (currently weight is 0 and hence it doesn't affect the results much)

nrnhines commented 3 years ago

tqperf for validation testing helped fix a bug #1533 I think it will be easy to extend to all the multisend variations. Testing would be less naive it was extended to verify event arrival and send times for each cell instead of just counts. The mod/invlfire needs a function like

shahash_add(string)

to incrementally add: gid, time, eventtype; everytime the cell receives and sends an event.