Skip to content

[Example] Fix reversed P2P direction logging#1808

Merged
cpcloud merged 1 commit intomainfrom
issue-1763
Mar 23, 2026
Merged

[Example] Fix reversed P2P direction logging#1808
cpcloud merged 1 commit intomainfrom
issue-1763

Conversation

@cpcloud
Copy link
Contributor

@cpcloud cpcloud commented Mar 23, 2026

Summary

  • Correct the reverse-direction peer access log in the cuda_bindings P2P examples.
  • Use j_access_i for GPU[j] -> GPU[i] so asymmetric peer access is reported accurately.

Test plan

  • cd cuda_bindings && pixi run pytest -q -s examples/0_Introduction/simpleP2P_test.py examples/extra/isoFDModelling_test.py
  • Validate the asymmetric direction logging path with a focused simulated 2-GPU run.

Closes #1763

Made with Cursor

Use the reverse peer-access check for GPU[j] -> GPU[i] so one-way P2P setups report the correct direction in both bindings examples.

Made-with: Cursor
@github-actions

This comment has been minimized.

@cpcloud cpcloud requested review from Andy-Jost and rparolin March 23, 2026 19:37
Copy link
Contributor

@Andy-Jost Andy-Jost left a comment

Choose a reason for hiding this comment

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

LGTM!

@cpcloud cpcloud merged commit b4a704c into main Mar 23, 2026
86 checks passed
@cpcloud cpcloud deleted the issue-1763 branch March 23, 2026 23:54
@github-actions
Copy link

Doc Preview CI
Preview removed because the pull request was closed or merged.

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]: P2P direction printing information checking error

2 participants