-
Notifications
You must be signed in to change notification settings - Fork 372
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.
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
Add a section in installation docs about running NEST with EBRAINS #3121
Add a section in installation docs about running NEST with EBRAINS #3121
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jessica-mitchell Looks good, just a few small suggestions.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall a good step forward for the documentation.
Only have a few comments.
Co-authored-by: Hans Ekkehard Plesser <[email protected]>
Co-authored-by: Hans Ekkehard Plesser <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
This PR, as discussed in docathon with @clinssen @steffengraber @heplesser @ddahmen , adds the try it on ebrains button for an installation free option in the install section.
Note that the links directly takes the person to a one neuron example (assuming they are logged and have server running). The reason: all the examples will be loaded, and the kernel that contains NEST should be set. So less changes for user.
We would need to add additional instructions if we want to open a plain notebook on EBRAINS, which is another possibility.
Missing from the instructions so far (wip in another branch, but can be added here) is this issues that you have to basically click twice to load an example if you dont have a server running yet.
#3066 #3067 should be merged first!
See output for Read the Docs here