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

Improve performance analysis tool #330

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from
Open

Improve performance analysis tool #330

wants to merge 6 commits into from

Conversation

ldowen
Copy link
Collaborator

@ldowen ldowen commented Feb 7, 2025 •
edited
Loading

Summary

  • This PR is an improvement to an existing tool
  • It does the following:
    • Modifies the performance analysis tool for better use and fixes bugs.
    • Changes the SPHERAL_TEST_INSTALL_PREFIX CMake variable to be relative.
    • Fixes and updates the documentation.

ToDo :

  • Annotate RELEASE_NOTES.md with notable changes.
  • Create LLNLSpheral PR pointing at this branch. (PR#128)
  • LLNLSpheral PR has passed all tests.

@ldowen ldowen requested review from jmikeowen and mdavis36 February 7, 2025 22:33
@ldowen ldowen self-assigned this Feb 7, 2025
@ldowen
Fixed SPHERAL_TEST_INSTALL_PREFIX, added advance region to use for pe…
8ae72c9 
…rformance regression testing, modified performance_analysis so it can be imported if necessary, added TIME_SCOPE and TIME_PHASE macros for Caliper timers
@ldowen
Merge branch 'develop' into feature/improve_perf_analysis
0e4a059
@ldowen
Try setting mpi4py version to 4.0.2
4b1e759
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jmikeowen jmikeowen Awaiting requested review from jmikeowen

@mdavis36 mdavis36 Awaiting requested review from mdavis36

At least 1 approving review is required to merge this pull request.

Assignees

@ldowen ldowen

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@ldowen