Skip to content

Update link for physicsnemo.models documentation#1227

Merged
coreyjadams merged 3 commits intomainfrom
ram-cherukuri-patch-4
Mar 10, 2026
Merged

Update link for physicsnemo.models documentation#1227
coreyjadams merged 3 commits intomainfrom
ram-cherukuri-patch-4

Conversation

@ram-cherukuri
Copy link
Copy Markdown
Collaborator

PhysicsNeMo Pull Request

Description

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.
  • An issue is linked to this pull request.

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.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Nov 12, 2025

Greptile Overview

Greptile Summary

Updated the physicsnemo.models documentation link to use the new URL structure (docs.nvidia.com/physicsnemo/latest/) instead of the old structure (docs.nvidia.com/deeplearning/physicsnemo/), and added a secondary "More Details" link to the API documentation.

  • Changed primary link from physicsnemo-core/api/physicsnemo.models.html to user-guide/model_architectures.html
  • Added additional reference link to physicsnemo/api_models.html for detailed API documentation
  • Other documentation links in the README still use the old URL structure and may need similar updates in future PRs

Important Files Changed

File Analysis

Filename Score Overview
README.md 5/5 Updated physicsnemo.models documentation link to point to new URL structure with additional reference link

Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

1 file reviewed, no comments

Edit Code Review Agent Settings | Greptile

@ktangsali
Copy link
Copy Markdown
Collaborator

/blossom-ci

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 1, 2026

This pull request is stale because it has been open for 90 days with no activity. Remove the stale label or comment to keep it open, otherwise this will be closed in 14 days.

@github-actions github-actions Bot added the stale label Mar 1, 2026
@coreyjadams
Copy link
Copy Markdown
Collaborator

/blossom-ci

@coreyjadams coreyjadams added this pull request to the merge queue Mar 10, 2026
Merged via the queue into main with commit a92a89d Mar 10, 2026
4 checks passed
@peterdsharpe peterdsharpe deleted the ram-cherukuri-patch-4 branch March 16, 2026 16:12
nbren12 pushed a commit to nbren12/modulus that referenced this pull request Mar 24, 2026
* Update link for physicsnemo.models documentation

* Update Model Zoo link in README.md

---------

Co-authored-by: Corey adams <6619961+coreyjadams@users.noreply.github.com>
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.

3 participants