Skip to content

Conversation

@heavyrain-lzy
Copy link
Contributor

@heavyrain-lzy heavyrain-lzy commented Mar 12, 2024

PR types

Others

PR changes

Others

Description

Pcard-76459
Add other options for dist.Strategy.

@paddle-bot
Copy link

paddle-bot bot commented Mar 12, 2024

你的PR提交成功,感谢你对开源项目的贡献!
请关注后续CI自动化测试结果,详情请参考Paddle-CI手册
Your PR has been submitted. Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

@heavyrain-lzy heavyrain-lzy force-pushed the complete_convert_strategy branch from 4a0f073 to 832a797 Compare March 12, 2024 13:23
@heavyrain-lzy
Copy link
Contributor Author

)
self._sp_optimization = auto_strategy.SPOptimizationConfig(config_dict)

def _convert_to_unify_strategy(self, auto_stragety):
Copy link
Contributor

Choose a reason for hiding this comment

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

-> _from_legacy_strategy()

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done. Thanks~

Copy link
Contributor

@zhiqiu zhiqiu left a comment

Choose a reason for hiding this comment

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

LGTM

@heavyrain-lzy heavyrain-lzy reopened this Mar 14, 2024
@heavyrain-lzy heavyrain-lzy merged commit b92fd61 into PaddlePaddle:develop Mar 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants