Skip to content

Geot docs update - #1895

Merged
coreyjadams merged 14 commits into
NVIDIA:mainfrom
coreyjadams:geot-docs-update
Aug 11, 2026
Merged

Geot docs update#1895
coreyjadams merged 14 commits into
NVIDIA:mainfrom
coreyjadams:geot-docs-update

Conversation

@coreyjadams

Copy link
Copy Markdown
Collaborator

PhysicsNeMo Pull Request

This PR catches some residual docs updates from the GeoTransolver migration. It should merge before code freeze but it is not blocking anyone, as far as I'm aware.

Description

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-pr-bot Bot commented Aug 5, 2026

Copy link
Copy Markdown

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.

@github-actions

github-actions Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

CODEOWNERS review map

Current for commit 5eaa1473b413. An approval covers every file listed for that owner; one owner is sufficient for shared files.

@ktangsali — 1 file(s)
  • docs/api/models/geotransolver.rst
@megnvidia — 1 file(s)
  • docs/api/models/geotransolver.rst

No CODEOWNER

  • CHANGELOG.md

Comment /codeowners-info to refresh.

@greptile-apps

greptile-apps Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

The PR refreshes GeoTransolver documentation following its migration from the experimental namespace.

  • Clarifies the changelog guidance for attaching the decoupled out-of-distribution guard.
  • Adds links to the GeoTransolver and FLARE papers.
  • Documents the GALE_FA attention backend and its API class.

Important Files Changed

Filename Overview
CHANGELOG.md Clarifies the promoted GeoTransolver API and accurately directs users to the exported guard wrapper and attachment helper.
docs/api/models/geotransolver.rst Adds valid paper references and accurate API documentation for the existing GALE_FA FLARE backend.

Reviews (1): Last reviewed commit: "Merge branch 'main' into geot-docs-updat..." | Re-trigger Greptile

@peterdsharpe peterdsharpe left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Nice

Comment thread docs/api/models/geotransolver.rst Outdated
Comment thread docs/api/models/geotransolver.rst Outdated
Comment thread CHANGELOG.md Outdated
Comment thread CHANGELOG.md Outdated
Comment thread CHANGELOG.md Outdated
Comment thread CHANGELOG.md Outdated
Co-authored-by: megnvidia <mmiranda@nvidia.com>
@coreyjadams
coreyjadams enabled auto-merge August 10, 2026 14:27
@coreyjadams

Copy link
Copy Markdown
Collaborator Author

/ok to test 8600de3

@coreyjadams coreyjadams added the ! - Release PRs or Issues releating to a release label Aug 11, 2026

@ktangsali ktangsali left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

These look good to me!

@coreyjadams

Copy link
Copy Markdown
Collaborator Author

/ok to test e536d80

@coreyjadams
coreyjadams added this pull request to the merge queue Aug 11, 2026
Merged via the queue into NVIDIA:main with commit acdf9d4 Aug 11, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

! - Release PRs or Issues releating to a release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants