Skip to content

Conversation

@dssgabriel
Copy link
Collaborator

Also, add the associated unit test.

Based on #192, which will need to be merged first.

@dssgabriel dssgabriel self-assigned this Dec 1, 2025
@dssgabriel dssgabriel added C-enhancement Category: an enhancement or bug fix A-core Area: KokkosComm core library implementation A-unit-tests Area: KokkosComm unit tests labels Dec 1, 2025
Copy link
Member

@cedricchevalier19 cedricchevalier19 left a comment

Choose a reason for hiding this comment

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

Looks ok.

Copy link
Member

@cedricchevalier19 cedricchevalier19 left a comment

Choose a reason for hiding this comment

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

We should add this in the documentation.

@dssgabriel dssgabriel force-pushed the feature/core-allgather branch from b06127e to 4791424 Compare December 4, 2025 08:07
@cwpearson cwpearson added the SNL-CI-APPROVAL Required to run SNL CI on non-SNL contributions label Dec 4, 2025
Also add associated unit test.

Signed-off-by: Gabriel Dos Santos <[email protected]>
Signed-off-by: Cedric Chevalier <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-core Area: KokkosComm core library implementation A-unit-tests Area: KokkosComm unit tests C-enhancement Category: an enhancement or bug fix SNL-CI-APPROVAL Required to run SNL CI on non-SNL contributions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants