Tutorial - #1439
Open
Yurlungur wants to merge 29 commits into
Open
Conversation
Yurlungur
requested review from
acreyes,
c-prather,
jdolence,
lroberts36,
pdmullen and
pgrete
August 24, 2026 00:42
Collaborator
Author
|
@par-hermes format |
Yurlungur
commented
Aug 24, 2026
| @@ -1,4 +1,4 @@ | |||
| .. par_for: | |||
| .. _par_for: | |||
Collaborator
Author
There was a problem hiding this comment.
This was a typo.
| plt.colorbar( | ||
| pm, label=colorbar, fontsize=fontsize, fraction=0.02, pad=0.04, ax=p | ||
| ) | ||
| plt.colorbar(pm, label=colorbar, fraction=0.02, pad=0.04, ax=p) |
Collaborator
Author
There was a problem hiding this comment.
matplotlib api changed. fontsize can't go here anymore.
12 tasks
Collaborator
Author
|
Tutorial live here: https://parthenon-hpc-lab.github.io/parthenon/jmm/turorial/src/tutorial.html |
Yurlungur
enabled auto-merge
August 24, 2026 16:29
pgrete
reviewed
Aug 24, 2026
pgrete
reviewed
Aug 24, 2026
Collaborator
Author
|
@pgrete @c-prather please re-review. It should be updated with your suggestions and the issues we found in the live run. |
c-prather
approved these changes
Sep 2, 2026
c-prather
left a comment
c-prather
left a comment
Contributor
There was a problem hiding this comment.
Left the changes I had to (one I wanted to) make when updating the compile-able example, and some wording things when going through the text again.
Otherwise LGTM, really excellent stuff and I'm glad we finally have this. Bravo.
Co-authored-by: Cora Prather <cprather@radimant.org>
Co-authored-by: Cora Prather <cprather@radimant.org>
Co-authored-by: Cora Prather <cprather@radimant.org>
Co-authored-by: Cora Prather <cprather@radimant.org>
Co-authored-by: Cora Prather <cprather@radimant.org>
… into jmm/turorial
Yurlungur
disabled auto-merge
September 2, 2026 20:21
Yurlungur
enabled auto-merge
September 2, 2026 20:21
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
7 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR Summary
In preparation for the Parthenon workshops coming up I finally sat down and wrote a tutorial for how to use parthenon. In the end I did not use gen AI as I wanted to convey very specifically the story I wanted to convey. I also made some minor tweaks to Parthenon guts and scripts to make sure the tutorial worked as intended.
I can't include the project I wrote that the tutorial walks through, as it would be a bit recursive. But if someone else wanted to create it in the parthenon namespace and follow the tutorial so we had a complete version, that wouldn't be unwelcome. That would be a good test my instructions work for someone else.
PR Checklist