Skip to content

Notebooks Cleanup#1674

Open
peterdsharpe wants to merge 5 commits into
NVIDIA:mainfrom
peterdsharpe:psharpe/notebook-cleanup
Open

Notebooks Cleanup#1674
peterdsharpe wants to merge 5 commits into
NVIDIA:mainfrom
peterdsharpe:psharpe/notebook-cleanup

Conversation

@peterdsharpe
Copy link
Copy Markdown
Collaborator

PhysicsNeMo Pull Request

Description

Strips "execution count" from IPYNB jsons for display cleanliness.

Checklist

Dependencies

Review Process

All PRs are reviewed by the PhysicsNeMo team before merging.

Depending on which files are changed, GitHub may automatically assign a maintainer for review.

We are also testing AI-based code review tools (e.g., Greptile), which may add automated comments with a confidence score.
This score reflects the AI’s assessment of merge readiness and is not a qualitative judgment of your work, nor is
it an indication that the PR will be accepted / rejected.

AI-generated feedback should be reviewed critically for usefulness.
You are not required to respond to every AI comment, but they are intended to help both authors and reviewers.
Please react to Greptile comments with 👍 or 👎 to provide feedback on their accuracy.

@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot Bot commented May 27, 2026

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@peterdsharpe
Copy link
Copy Markdown
Collaborator Author

/blossom-ci

@peterdsharpe peterdsharpe requested a review from coreyjadams May 27, 2026 15:10
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 27, 2026

Greptile Summary

This PR nullifies execution_count fields (setting them from non-null integers to null) across six mesh tutorial notebooks for cleaner display in version control diffs. All outputs are preserved unchanged.

  • All six notebooks also have their recorded Python kernel version changed from 3.13.8 to 3.12.12, which is not mentioned in the PR description and appears to be a side effect of re-saving the notebooks in a different local environment.
  • The execution count cleanup itself is idiomatic and follows standard notebook hygiene practices.

Important Files Changed

Filename Overview
examples/minimal/mesh/tutorial_1_getting_started.ipynb Sets 6 execution_count values to null and downgrades recorded kernel version from 3.13.8 to 3.12.12
examples/minimal/mesh/tutorial_2_operations.ipynb Sets 1 execution_count to null and downgrades recorded kernel version from 3.13.8 to 3.12.12
examples/minimal/mesh/tutorial_3_calculus.ipynb Sets 5 execution_count values to null and downgrades recorded kernel version from 3.13.8 to 3.12.12
examples/minimal/mesh/tutorial_4_neighbors_spatial.ipynb Sets 1 execution_count to null and downgrades recorded kernel version from 3.13.8 to 3.12.12
examples/minimal/mesh/tutorial_5_quality_repair.ipynb Sets 3 execution_count values to null and downgrades recorded kernel version from 3.13.8 to 3.12.12
examples/minimal/mesh/tutorial_6_ml_integration.ipynb Sets 4 execution_count values to null and downgrades recorded kernel version from 3.13.8 to 3.12.12

Reviews (1): Last reviewed commit: "removes execution count for cleanliness" | Re-trigger Greptile

Comment thread examples/minimal/mesh/tutorial_1_getting_started.ipynb
@peterdsharpe peterdsharpe requested a review from mnabian May 27, 2026 15:17
@peterdsharpe peterdsharpe enabled auto-merge May 27, 2026 15:17
@peterdsharpe
Copy link
Copy Markdown
Collaborator Author

/blossom-ci

@peterdsharpe
Copy link
Copy Markdown
Collaborator Author

/blossom-ci

@peterdsharpe
Copy link
Copy Markdown
Collaborator Author

/ok to test 4332a04

@peterdsharpe
Copy link
Copy Markdown
Collaborator Author

/blossom-ci

@peterdsharpe
Copy link
Copy Markdown
Collaborator Author

/ok to test 87b1e0d

@peterdsharpe
Copy link
Copy Markdown
Collaborator Author

/blossom-ci

@peterdsharpe
Copy link
Copy Markdown
Collaborator Author

/blossom-ci

@peterdsharpe
Copy link
Copy Markdown
Collaborator Author

/ok to test 09344d9

@peterdsharpe
Copy link
Copy Markdown
Collaborator Author

/blossom-ci

@peterdsharpe
Copy link
Copy Markdown
Collaborator Author

/ok to test 2b5e754

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