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.

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 connectivity diagram to hpc_benchmark documentation #2247

Merged
merged 4 commits into from
Dec 15, 2021

Conversation

jarsi
Copy link
Contributor

@jarsi jarsi commented Dec 14, 2021

During the review phase of a manuscript we realized that it might be beneficial to have figures of the most important models as part of the user level documentation. This PR graphically extends the user level documentation of the hpc_benchmark.py file with a detailed figure of its connectivity. The figure and its description are taken (and slightly adapted) from Senk et al (2021).

During integration of the figure we realized that the copying and linking of figures requires long relative paths, this can be addressed by a future PR, @terhorstd.

I suggest @jhnnsnk as a reviewer.

@terhorstd terhorstd added S: Normal Handle this with default priority T: Enhancement New functionality, model or documentation labels Dec 15, 2021
@terhorstd terhorstd added this to In progress in Documentation via automation Dec 15, 2021
@terhorstd terhorstd added this to To do in Models via automation Dec 15, 2021
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.

Thanks a lot for working on the content!
Spread the word, that this kind of small enhancements/fixes are highly appreciated! We need more of these!
🚀

Documentation automation moved this from In progress to Review Dec 15, 2021
@terhorstd terhorstd changed the title Extend hpc benchmark documentation Extend hpc_benchmark documentation Dec 15, 2021
@terhorstd
Copy link
Contributor

@jhnnsnk, I also added you for the content side review.

Copy link
Contributor

@jougs jougs left a comment

Choose a reason for hiding this comment

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

Just some minor suggestions from my side. The ones on the SVG file remove the diagonal lines through the "A"s next to the feedback connections, as they are not really explained in this context.

pynest/examples/hpc_benchmark.py Outdated Show resolved Hide resolved
pynest/examples/hpc_benchmark.py Outdated Show resolved Hide resolved
jarsi and others added 2 commits December 15, 2021 14:26
Co-authored-by: Jochen Martin Eppler <jougs@gmx.net>
Co-authored-by: Jochen Martin Eppler <jougs@gmx.net>
pynest/examples/hpc_benchmark.py Outdated Show resolved Hide resolved
Co-authored-by: Jochen Martin Eppler <jougs@gmx.net>
@jougs jougs changed the title Extend hpc_benchmark documentation Add connectivity diagram to hpc_benchmark documentation Dec 15, 2021
@jougs jougs merged commit f91917b into nest:master Dec 15, 2021
Documentation automation moved this from Review to Done Dec 15, 2021
@clinssen clinssen moved this from To do to Done in Models Nov 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S: Normal Handle this with default priority T: Enhancement New functionality, model or documentation
Projects
Documentation
  
Done
Models
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants