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

Update MAM benchmark on website #3358

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Update MAM benchmark on website #3358

wants to merge 3 commits into from

Conversation

mlober
Copy link
Contributor

@mlober mlober commented Nov 5, 2024

This PR corrects the benchmark results for the multi-area model with NEST 3.8 on the NEST website. It separates the strong scaling plot into ground and metastable state.

@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 Nov 5, 2024
@jessica-mitchell
Copy link
Contributor

Link to Read the docs build: https://nest-simulator--3358.org.readthedocs.build/en/3358/benchmark_results.html

jessica-mitchell
jessica-mitchell requested changes Nov 5, 2024
View reviewed changes
Copy link
Contributor

@jessica-mitchell jessica-mitchell left a comment

Choose a reason for hiding this comment

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

Images are not rendering- I think it's the lack of new line, added it as suggestion

doc/htmldoc/benchmark_results.rst Show resolved Hide resolved
doc/htmldoc/benchmark_results.rst Show resolved Hide resolved
mlober and others added 2 commits November 5, 2024 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jessica-mitchell jessica-mitchell jessica-mitchell requested changes

Requested changes must be addressed to merge this pull request.

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
Status: Review
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mlober @jessica-mitchell