Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Graph.to_str() 的 shuffle 参数不适用于有根树 #101

Open
Nanarikom opened this issue Jan 23, 2023 · 1 comment
Open

Graph.to_str() 的 shuffle 参数不适用于有根树 #101

Nanarikom opened this issue Jan 23, 2023 · 1 comment

Comments

@Nanarikom
Copy link

有根树使用 Graph.to_str() 时,如果使用 shuffle=True 参数,会导致根结点被 shuffle 到其它位置,这一操作对于有根树显然是没有任何意义的。为了避开 cyaron 的 shuffle 操作而手写 shuffle 的繁琐程度已经超过了直接用 Cpp 手搓数据。我建议参数应当允许仅 shuffle 点或仅 shuffle 边(或者支持不 shuffle 根节点的功能)。

@weilycoder
Copy link
Contributor

提交了 pr,允许你自定义打乱点和边的函数,你可以在函数里获取根被映射到的位置,或者不对点进行打乱。

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

No branches or pull requests

2 participants