Skip to content

Conversation

@msimberg
Copy link
Collaborator

@msimberg msimberg commented Nov 5, 2025

No description provided.

@msimberg msimberg requested a review from Madeeks as a code owner November 5, 2025 08:15
@msimberg msimberg requested a review from jpcoles-cscs November 5, 2025 08:16
@github-actions

This comment has been minimized.

@github-actions
Copy link

github-actions bot commented Nov 5, 2025

preview available: https://docs.tds.cscs.ch/295

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions
Copy link

github-actions bot commented Nov 5, 2025

preview available: https://docs.tds.cscs.ch/295

@github-actions

This comment has been minimized.

@msimberg msimberg force-pushed the nccl-nchannels-per-net-peer branch from c1a1417 to 4d95323 Compare November 5, 2025 09:04
@github-actions
Copy link

github-actions bot commented Nov 5, 2025

preview available: https://docs.tds.cscs.ch/295

Copy link
Collaborator

@jpcoles-cscs jpcoles-cscs left a comment

Choose a reason for hiding this comment

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

This envvar is also discussed in the linked paper.
Maybe we could say:
"The option is officially undocumented, but ... and the paper linked above contain additional details."

@msimberg
Copy link
Collaborator Author

msimberg commented Nov 5, 2025

This envvar is also discussed in the linked paper. Maybe we could say: "The option is officially undocumented, but ... and the paper linked above contain additional details."

Good idea: bea05e1.

@github-actions
Copy link

github-actions bot commented Nov 5, 2025

preview available: https://docs.tds.cscs.ch/295

Copy link
Collaborator

@jpcoles-cscs jpcoles-cscs left a comment

Choose a reason for hiding this comment

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

Looks good now.

@bcumming bcumming added this pull request to the merge queue Nov 5, 2025
Merged via the queue into eth-cscs:main with commit 23fceb4 Nov 5, 2025
3 checks passed
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.

4 participants