Skip to content

fix: broken links in README's Suite of Physics-Informed ML Models section#1578

Open
Brumbelow wants to merge 1 commit intoNVIDIA:mainfrom
Brumbelow:fix/readme-suite-model-links
Open

fix: broken links in README's Suite of Physics-Informed ML Models section#1578
Brumbelow wants to merge 1 commit intoNVIDIA:mainfrom
Brumbelow:fix/readme-suite-model-links

Conversation

@Brumbelow
Copy link
Copy Markdown

@Brumbelow Brumbelow commented Apr 21, 2026

Summary

Fixed REAME.md links.

Motivation

Fixes #1303

Checklist

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Apr 21, 2026

Greptile Summary

This PR replaces seven dead docs.nvidia.com links in the "A Suite of Physics-Informed ML Models" section of the top-level README.md with working in-repo GitHub URLs, and adds a corresponding CHANGELOG.md entry. All seven new link targets were verified to exist in the repository (e.g. physicsnemo/nn/module/gnn_layers, examples/cfd/vortex_shedding_mgn, examples/weather/corrdiff, etc.).

Important Files Changed

Filename Overview
README.md Seven broken links in the "A Suite of Physics-Informed ML Models" section replaced with valid in-repo GitHub paths; all new targets verified to exist in the repository.
CHANGELOG.md Added a changelog entry for the README link fix under the correct section; wording is accurate and closes the referenced issue.

Reviews (1): Last reviewed commit: "Fix broken links in README's Suite of Ph..." | Re-trigger Greptile

@Brumbelow Brumbelow changed the title Fix broken links in README's Suite of Physics-Informed ML Models section (#1303) Fix broken links in README's Suite of Physics-Informed ML Models section Apr 21, 2026
@Brumbelow Brumbelow changed the title Fix broken links in README's Suite of Physics-Informed ML Models section fix: broken links in README's Suite of Physics-Informed ML Models section Apr 21, 2026
@Brumbelow Brumbelow force-pushed the fix/readme-suite-model-links branch from b11896d to 70d688d Compare April 21, 2026 12:53
@coreyjadams coreyjadams enabled auto-merge April 21, 2026 17:34
…ion (NVIDIA#1303)

Signed-off-by: Andrew Brumbelow <andrewbrumbelow@gmail.com>
auto-merge was automatically disabled April 21, 2026 18:02

Head branch was pushed to by a user without write access

@Brumbelow Brumbelow force-pushed the fix/readme-suite-model-links branch from 70d688d to cd55a0b Compare April 21, 2026 18:02
@Brumbelow Brumbelow requested a review from coreyjadams April 21, 2026 18:02
@Brumbelow
Copy link
Copy Markdown
Author

Sorry @coreyjadams not sure what happened. Didn't mean to rebase. Leaving alone now

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.

🐛[BUG]: Links of Physics-Informed ML Models in README broken

2 participants