Skip to content

Conversation

@pbdanny
Copy link

@pbdanny pbdanny commented Oct 19, 2025

No description provided.

Signed-off-by: Danny Thanakrit Boonquarmdee <[email protected]>
@kunwuz
Copy link
Collaborator

kunwuz commented Oct 21, 2025

Thanks so much for the contribution! Could you please elaborate a bit more on the context of this PR? For example, what's the motivation for including relabelling for nx_graph?

@pbdanny
Copy link
Author

pbdanny commented Oct 21, 2025 via email

@kunwuz
Copy link
Collaborator

kunwuz commented Oct 22, 2025

Aha, thanks for the careful thought. Since later we use integers to add edges, perhaps applying relabeling before that can unintentionally mix integer and string node types, which may lead to potential inconsistency. The existing self.labels mappings already provide a way to reference node names if needed, so we may want to keep the graph structure unchanged here.

@pbdanny
Copy link
Author

pbdanny commented Oct 22, 2025 via email

@kunwuz kunwuz closed this Oct 23, 2025
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