Skip to content

Updates for compatibility with PhysicsNeMo 2.0+#39

Merged
peterdsharpe merged 2 commits into
NVIDIA:mainfrom
peterdsharpe:psharpe/migrate-to-pnm2
May 22, 2026
Merged

Updates for compatibility with PhysicsNeMo 2.0+#39
peterdsharpe merged 2 commits into
NVIDIA:mainfrom
peterdsharpe:psharpe/migrate-to-pnm2

Conversation

@peterdsharpe
Copy link
Copy Markdown
Collaborator

PhysicsNeMo-CFD Pull Request

Description

  • Clarified test skipping conditions in CONTRIBUTING.md for compute_error_vs_sdf.
  • Refactored design_datapipe.py to replace deprecated utility imports and introduced numpy wrappers for SDF and grid creation functions to maintain compatibility with the new torch-only implementations.
  • Updated main.py to reflect changes in utility imports.

Checklist

  • I am familiar with the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.
  • The CHANGELOG.md is up to date with these changes.

Dependencies

…ity with PhysicsNeMo v2

- Clarified test skipping conditions in CONTRIBUTING.md for `compute_error_vs_sdf`.
- Refactored `design_datapipe.py` to replace deprecated utility imports and introduced numpy wrappers for SDF and grid creation functions to maintain compatibility with the new torch-only implementations.
- Updated `main.py` to reflect changes in utility imports.
@peterdsharpe peterdsharpe requested a review from ktangsali May 20, 2026 18:36
@peterdsharpe
Copy link
Copy Markdown
Collaborator Author

@abokov-nv for awareness, this closes NVBUGS 6188548

Copy link
Copy Markdown
Collaborator

@ktangsali ktangsali left a comment

Choose a reason for hiding this comment

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

LGTM. Quick question - why is this PR changing the uv.lock file? Is that because it was never updated?

Comment thread workflows/domino_design_sensitivities/design_datapipe.py
@peterdsharpe
Copy link
Copy Markdown
Collaborator Author

/blossom-ci

@peterdsharpe
Copy link
Copy Markdown
Collaborator Author

Also: this PR adds the uv.lock file, since none existed before. This is intended to help reproducibility for end-users.

@peterdsharpe peterdsharpe enabled auto-merge May 22, 2026 00:46
@peterdsharpe peterdsharpe added this pull request to the merge queue May 22, 2026
Merged via the queue into NVIDIA:main with commit 3d94d44 May 22, 2026
3 checks passed
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