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.

Sign up for GitHub

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

Jump to bottom

Include NEST GPU into the documentation #2418

Merged
merged 2 commits into from
Jun 27, 2022
Merged

Include NEST GPU into the documentation #2418

merged 2 commits into from
Jun 27, 2022

Conversation

jessica-mitchell
Copy link
Contributor

This PR adds the NEST GPU repo as an intersphinx link, and also adds a link in the related projects page.

@jessica-mitchell jessica-mitchell changed the title Include NEST GPU to to docs Include NEST GPU to the docs Jun 23, 2022
@jessica-mitchell jessica-mitchell added S: Normal Handle this with default priority T: Maintenance Work to keep up the quality of the code and documentation. I: No breaking change Previously written code will work as before, no one should note anything changing (aside the fix) labels Jun 23, 2022
@jessica-mitchell jessica-mitchell added this to In progress in Documentation via automation Jun 23, 2022
@jessica-mitchell jessica-mitchell changed the title Include NEST GPU to the docs Include NEST GPU into the docs Jun 23, 2022
@ikitayama
Copy link

Hi where can I see the GPU code? Written in CUDA C++ or pure C++ with OpenMP offloading?

@jessica-mitchell
Copy link
Contributor Author

@ikitayama I think you can find what you need here: https://github.com/nest/nest-gpu but @jhnnsnk or @JoseJVS may be able to tell you more.

@jhnnsnk
Copy link
Contributor

jhnnsnk commented Jun 23, 2022

@ikitayama CUDA C++ is correct. @jessica-mitchell already posted the repository, here is in addition the current documentation (under development): https://nest-gpu.readthedocs.io

@ikitayama
Copy link

Thanks! I've been working on OepnMP offloading for NEST I'll take a look.

@ikitayama
Copy link

Thanks! I've been working on OepnMP offloading for NEST I'll take a look.

... and so far, not quite successful due to the lack of the C++ runtime library support on device.

terhorstd
terhorstd approved these changes Jun 27, 2022
View reviewed changes
Copy link
Contributor

@terhorstd terhorstd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

Documentation automation moved this from In progress to Review Jun 27, 2022
@terhorstd
Copy link
Contributor

Waiting for rerun to finish, but error was unrelated. If problem persists, it should go to a separate issue and be handled separately.

@terhorstd terhorstd mentioned this pull request Jun 27, 2022
Closed
@terhorstd
Copy link
Contributor

Created issue #2425 to follow up on the unrelated issue with the numerics.
Merging this one, as it passes all other checks.

@terhorstd terhorstd merged commit c266e5c into nest:master Jun 27, 2022
Documentation automation moved this from Review to Done Jun 27, 2022
@terhorstd terhorstd changed the title Include NEST GPU into the docs Include NEST GPU into the documentation Jul 20, 2022
@jessica-mitchell jessica-mitchell deleted the add-gpu-proj branch April 17, 2024 18:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@terhorstd terhorstd terhorstd approved these changes

Assignees
No one assigned
Labels
I: No breaking change Previously written code will work as before, no one should note anything changing (aside the fix) S: Normal Handle this with default priority T: Maintenance Work to keep up the quality of the code and documentation.
Projects
Documentation
  
Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@jessica-mitchell @ikitayama @jhnnsnk @terhorstd