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

【Hackathon 5th No.64】PaddleSeg套件模型接入动转静训练功能-part #3592

Merged
merged 5 commits into from
Dec 12, 2023

Conversation

huajiao-hjyp
Copy link
Contributor

PR types

Others

PR changes

Others

Description

PaddleSeg套件模型接入动转静训练功能

Copy link

paddle-bot bot commented Dec 8, 2023

Thanks for your contribution!

@paddle-bot paddle-bot bot added the contributor Contribution from developers label Dec 8, 2023
@2742195759
Copy link
Contributor

是否可以在列表后面或者是最后贴上命令(动态图 + 动转静)方便后续复现。

@huajiao-hjyp
Copy link
Contributor Author

是否可以在列表后面或者是最后贴上命令(动态图 + 动转静)方便后续复现。

已附上命令,因为to_static_training参数是通过cfg.to_static_training获取的,所以要在xxx.yaml配置文件中进行参数的配置。

@2742195759
Copy link
Contributor

是这样的,后续我们会将这些模型接入到一个例行监控平台,自动平台可无法支持修改一个yaml然后在改回来,会有不方便的过程。所以我们希望一个命令直接可以运行动转静。

  1. 可以check一下train.py中是否存在覆写config中字段的方式(应该有),然后将 to_static_training 的字段通过覆写的方式放入到命令行中,比如类似:
python train.py xxxxxx -o Global.to_static_training=True 

来开启动转静训练。

  1. 如果没有的话,可以在 train 中写一个函数来支持覆写逻辑。

麻烦同学按照上述的要求,修改一下命令行吧。如果需要支持覆写也麻烦支持一下。感谢~~~

@huajiao-hjyp
Copy link
Contributor Author

是这样的,后续我们会将这些模型接入到一个例行监控平台,自动平台可无法支持修改一个yaml然后在改回来,会有不方便的过程。所以我们希望一个命令直接可以运行动转静。

  1. 可以check一下train.py中是否存在覆写config中字段的方式(应该有),然后将 to_static_training 的字段通过覆写的方式放入到命令行中,比如类似:
python train.py xxxxxx -o Global.to_static_training=True 

来开启动转静训练。

  1. 如果没有的话,可以在 train 中写一个函数来支持覆写逻辑。

麻烦同学按照上述的要求,修改一下命令行吧。如果需要支持覆写也麻烦支持一下。感谢~~~

好的,已修改~

因为PaddleSeg中是将配置转为了Config对象,所以修改了Config类的构造函数来支持覆写(不同于常规的合并两个dict)。

修改后既可以在配置文件中通过to_static_training: True配置,也可以在命令行通过--to_static_training标志开启静态图,命令行优先级更高。

Copy link
Contributor

@2742195759 2742195759 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@shiyutang shiyutang left a comment

Choose a reason for hiding this comment

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

LGTM

@shiyutang shiyutang merged commit 41cd8b6 into PaddlePaddle:release/2.9 Dec 12, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contributor Contribution from developers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants