Hans Ekkehard Plesser

Results 330 comments of Hans Ekkehard Plesser

> Ok I just saw that there was a merged PR regarding the unit test for STDP dopamine, and a few tests that reference it, so wasnt sure if it...

#1840 tests plain stdp, but not stdp_dopamine_synapse. I have searched through our testsuite and have not found any test for the correct dynamics of the dopamine synapse.

The cause of the problem appears to be that the SLI `Take` function preserves spatial metadata, while the new and correct solution implemented in #3632 uses boolean indexing here https://github.com/nest/nest-simulator/blob/cd8c487ee959eb0d87ca73d9996b914d8795727e/pynest/nest/lib/hl_api_nodes.py#L215...

I think we need to move `sli2py_music` out of the `pytests` directory, or we need to ignore also `sli2py_music` here: https://github.com/nest/nest-simulator/blob/84da3b5e044c4e1b8e482f3a079af904a23f8f9f/testsuite/do_tests.sh#L536 Otherwise, Pytest test collection finds the `test_....py` files and...

@JanVogelsang The music tests failed. From the downloaded artefacts it looks like the first one had too few MPI slots: ``` Running test 'test_cont_proxy.music' with 5 processes... > -------------------------------------------------------------------------- >...

@tammoippen Do you know if there is any news on this one?

This will be fixed by #3524 which eliminates the SLI interpreter.

@JanVogelsang This looks very interesting! Just to make sure I understand your solution correctly: Instead of having the individual managers as actual data members of the `KernelManager` instance, the `KernelManager`...

> I have no idea what's supposed to be wrong with the 2 copyright headers. @terhorstd Could you take a look please? Very strange. Have you tried to run the...