nest-simulator icon indicating copy to clipboard operation
nest-simulator copied to clipboard

Refactoring the startup.

Open tammoippen opened this issue 9 years ago • 6 comments

The function neststartup() in nest/neststartup.h/cpp is used both for starting up the sli-nest executable as well as the PyNEST module. A compile time flag _IS_PYNEST is given to compile this file to differentiate what is build. In master this is buried in pynest/Makefile.am, which uses the files in ../nest/ as sources.

In cmake (#213) a target nest_lib in nest/CMakeLists.txt is build. Depending on the availability of Python this is build with -D_IS_PYNEST and later used as a library to pynestkernel.

I propose, that we should refactor the method: only have NEST specific startup code inside it; then create some method sli_nest_startup(), which calls the neststartup() and initializes all sli relevant code and create some method pynest_nest_startup(), which calls the neststartup() and initializes all pytnest relevant code.

tammoippen avatar Mar 10 '16 10:03 tammoippen

Agreed, that would be much clearer and flexible

apeyser avatar Mar 10 '16 12:03 apeyser

see #853.

tammoippen avatar Nov 07 '17 13:11 tammoippen

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

heplesser avatar Sep 13 '18 09:09 heplesser

None from my side, sorry.

tammoippen avatar Sep 13 '18 16:09 tammoippen

Issue automatically marked stale!

github-actions[bot] avatar Sep 06 '21 08:09 github-actions[bot]

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

heplesser avatar Nov 24 '25 22:11 heplesser