Skip to content

Restore test_full_model_gp_correctness in test_predict.py - #2148

Merged
rayg1234 merged 2 commits into
mainfrom
rgao_restore_gp_test
Aug 11, 2026
Merged

Restore test_full_model_gp_correctness in test_predict.py#2148
rayg1234 merged 2 commits into
mainfrom
rgao_restore_gp_test

Conversation

@rayg1234

Copy link
Copy Markdown
Contributor

Re-add the full-model GP correctness test (no-GP / allgather / A2A-spatial / A2A-index_split at 1/2/4 workers vs single-GPU reference on energy/forces/ stress), which was silently dropped during an earlier main merge. Adapted to the current gp_config=GraphParallelConfig(...) API and the pretrained fixture convention. Referenced by graph_parallel_verification.md section 1d.

Re-add the full-model GP correctness test (no-GP / allgather / A2A-spatial /
A2A-index_split at 1/2/4 workers vs single-GPU reference on energy/forces/
stress), which was silently dropped during an earlier main merge. Adapted to
the current gp_config=GraphParallelConfig(...) API and the pretrained fixture
convention. Referenced by graph_parallel_verification.md section 1d.
@meta-cla meta-cla Bot added the cla signed label Aug 11, 2026
@rayg1234
rayg1234 requested a review from aditvenk August 11, 2026 05:27
@rayg1234 rayg1234 added enhancement New feature or request minor Minor version release patch Patch version release labels Aug 11, 2026

@aditvenk aditvenk left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks, verified with #2146 that the test passes.

@rayg1234
rayg1234 added this pull request to the merge queue Aug 11, 2026
Merged via the queue into main with commit a3fa357 Aug 11, 2026
26 checks passed
@rayg1234
rayg1234 deleted the rgao_restore_gp_test branch August 11, 2026 22:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla signed enhancement New feature or request minor Minor version release patch Patch version release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants