search

nest / nest-gpu

NEST GPU
https://nest-gpu.readthedocs.io
GNU General Public License v2.0
19 stars 12 forks source link

CMake Implementation #45

Closed JoseJVS closed 1 year ago

JoseJVS commented 2 years ago

First working CMake implementation, with all tests succeeding. Still in draft as testing and documentation targets are not yet implemented (code commented out to serve as roadmap), however I think they would be more appropriate for another pull request. Furthermore, the configuration summary still needs to be re-implemented (but not a necessary feature).

jhnnsnk commented 2 years ago

Thank you very much, @JoseJVS! At least the documentation of how to install NEST GPU now should be updated as part of this PR, I think. Maybe @gmtiddia could help with that? Apart from that I agree that some features could become follow-up PRs. Maybe we could directly create new issues and move the respective files (the ones copied from plain NEST) from this PR to another draft PR?

gmtiddia commented 2 years ago

Sure, I'll help with the documentation update for this PR.

JoseJVS commented 2 years ago

With the configure summary module adapted to NEST GPU, the CMake implementation is now fully functional. There are still the documentation and testing targets that are not yet ready to be implemented but those can be worked on other PRs.

JoseJVS commented 1 year ago

Updated branch with suggested changes from reviewers