Skip to content

early out for non distributed arrays#2350

Draft
mtar wants to merge 7 commits into
mainfrom
2326-_Commmunication_Early_out_for_non-distributed_arrays
Draft

early out for non distributed arrays#2350
mtar wants to merge 7 commits into
mainfrom
2326-_Commmunication_Early_out_for_non-distributed_arrays

Conversation

@mtar

@mtar mtar commented Jun 9, 2026

Copy link
Copy Markdown
Collaborator

Due Diligence

  • General:
  • Implementation:
    • unit tests: all split configurations tested
    • unit tests: multiple dtypes tested
    • NEW unit tests: MPS tested (1 MPI process, 1 GPU)
    • benchmarks: created for new functionality
    • benchmarks: performance improved or maintained
    • documentation updated where needed

Description

The collective MPI communication functions check if the communicator size is one and copies the send buffer to the receive buffer if available. Additionally, this PR removes direct access to mpi variables outside of the communication module.

Issue/s resolved: #2326

Changes proposed:

  • access MPI variables through own communicator, not directly
  • make general communicator abstract
  • add early out in collective mpi communication functions
  • use newer type hints

Type of change

Refactor

Memory requirements

Performance

Does this change modify the behaviour of other functions? If so, which?

yes / no

@mtar mtar changed the title [commmunication] early out for non distributed arrays early out for non distributed arrays Jun 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

[Commmunication] Early out for non-distributed arrays

1 participant