Skip to content

Conversation

@co63oc
Copy link
Contributor

@co63oc co63oc commented Jan 29, 2024

PR types

Others

PR changes

Others

Description

Fix PADDLE_WITH_DITRIBUTE PADDLE_WITH_DISTRIBUTE

注释没有使用代码,如果开启CI错误

@paddle-bot paddle-bot bot added the contributor External developers label Jan 29, 2024
@luotao1 luotao1 added the HappyOpenSource 快乐开源活动issue与PR label Jan 30, 2024
luotao1
luotao1 previously approved these changes Jan 30, 2024
@luotao1 luotao1 self-assigned this Jan 30, 2024
REGISTER_PUBLIC_GLOBAL_VAR(FLAGS_rpc_get_thread_num);
REGISTER_PUBLIC_GLOBAL_VAR(FLAGS_rpc_prefetch_thread_num);
#endif
// #ifdef PADDLE_WITH_DISTRIBUTE
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please remove useless codes instead of commenting them.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

已修改

// #ifdef PADDLE_WITH_DISTRIBUTE
// REGISTER_PUBLIC_GLOBAL_VAR(FLAGS_rpc_get_thread_num);
// REGISTER_PUBLIC_GLOBAL_VAR(FLAGS_rpc_prefetch_thread_num);
// #endif
Copy link
Contributor

Choose a reason for hiding this comment

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

@sneaxiy 讨论后,这段注释掉的代码可以进行删除,辛苦 @co63oc 再改一下

Copy link
Contributor Author

Choose a reason for hiding this comment

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

已修改

@luotao1 luotao1 merged commit 743a2c9 into PaddlePaddle:develop Feb 3, 2024
@co63oc co63oc deleted the patch-2 branch February 18, 2024 23:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contributor External developers HappyOpenSource 快乐开源活动issue与PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants