Skip to content

Add ESMF profiling tool #39

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

Open
wants to merge 15 commits into
base: main
Choose a base branch
from
Open

Conversation

minghangli-uni
Copy link
Contributor

The ESMF Profiling tool is a Python-based tool designed to read and process performance profile data from ESMF profiling log files. It provides a structured way to extract hierachical timing and computational stats for various regions within ESMF runs, enabling detailed performance analysis.

See #38.

@minghangli-uni minghangli-uni force-pushed the add-ESMF_Profiling_iss38 branch from b58c16e to bd5a276 Compare October 31, 2024 04:58
@minghangli-uni
Copy link
Contributor Author

Hi @aekiss , when you are available, could you please review this PR?

@minghangli-uni minghangli-uni requested a review from aekiss October 31, 2024 05:12
@dougiesquire
Copy link
Collaborator

@minghangli-uni, what does this do differently than the esmf profiling tools in om3-utils? Is this a replacement for those tools or an extension of them? Or something else?

@minghangli-uni
Copy link
Contributor Author

esmf profiling tools in om3-utils is more comprehensive but needs the additional package bt2 to process the binary CTF output, while this tool processes text output instead. My vision is that integrating existing visualisation tools (such as Trace Compass) for CTF profiling makes a lot of sense, especially for users who want more intuitive performance analysis. If the goal is quick insights, text-based summaries might be enough. But for detailed performance tuning and debugging, visualisation is a game-changer.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants