Skip to content

examples: clean up unused members in context_t#1989

Closed
chenhengqi wants to merge 1 commit intoNVIDIA:masterfrom
chenhengqi:refactor-examples
Closed

examples: clean up unused members in context_t#1989
chenhengqi wants to merge 1 commit intoNVIDIA:masterfrom
chenhengqi:refactor-examples

Conversation

@chenhengqi
Copy link
Copy Markdown
Contributor

Description

Remove nccl_id, comm, nccl_unique_id and thread_ranks from context_t as they are no longer used. Clean up associated memory allocation and deallocation code.

Related Issues

N/A

Changes & Impact

N/A

Performance Impact

N/A

@teojgo
Copy link
Copy Markdown
Collaborator

teojgo commented Apr 3, 2026

@chenhengqi can you update this to align with the new examples path, under docs/examples ?

Remove nccl_id, comm, nccl_unique_id and thread_ranks from context_t
as they are no longer used. Clean up associated memory allocation
and deallocation code.

Signed-off-by: Hengqi Chen <hengqi.chen@gmail.com>
@chenhengqi
Copy link
Copy Markdown
Contributor Author

@teojgo done.

@teojgo
Copy link
Copy Markdown
Collaborator

teojgo commented Apr 7, 2026

/mirror

@teojgo
Copy link
Copy Markdown
Collaborator

teojgo commented Apr 10, 2026

@chenhengqi, this has been merged here 628c20a.
Thank you for your contribution!

@teojgo teojgo closed this Apr 10, 2026
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.

2 participants