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

fix bugs of ClipGradByGlobalNorm in HybridParallel #36555

Merged
merged 3 commits into from
Oct 20, 2021
Merged

fix bugs of ClipGradByGlobalNorm in HybridParallel #36555

merged 3 commits into from
Oct 20, 2021

Conversation

haohongxiang
Copy link
Contributor

@haohongxiang haohongxiang commented Oct 19, 2021

PR types

Bug fixes

PR changes

Others

Describe

fix bugs of ClipGradByGlobalNorm

单卡FP32与3D混合并行+recompute+FP16的loss曲线精度对比
891d18e235bdacc0553a268951a16a57

@haohongxiang haohongxiang changed the title fix bugs of ClipGradByGlobalNorm fix bugs of ClipGradByGlobalNorm in HybridParallel Oct 19, 2021
new_grad = layers.elementwise_mul(x=g, y=clip_var)
if p.dtype == paddle.float16:
new_grad = layers.elementwise_mul(
x=g, y=paddle.cast(clip_var, paddle.float16))
Copy link
Member

@ForFishes ForFishes Oct 19, 2021

Choose a reason for hiding this comment

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

这个可以先cast一次,不需要出现在循环里,每一次都cast。

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

Copy link
Member

@ForFishes ForFishes left a comment

Choose a reason for hiding this comment

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

LGTM

@ForFishes ForFishes merged commit 6a3941e into PaddlePaddle:develop Oct 20, 2021
@haohongxiang haohongxiang deleted the fix_bugs_of_clip_grad branch October 25, 2021 06:36
fuyinno4 pushed a commit that referenced this pull request Oct 26, 2021
…imizer (#36707)

* fix bugs in HybridParallelClipGrad of hybrid_parallel_optimizer (#36237)

* fix bugs in HybridParallelClipGrad of hybrid_parallel_optimizer

* update

* update

* fix bugs in mp_layers、pp_layers and HybridParallelClipGrad (#36144)

* fix calling bug of HybridParallelClipGrad

* fix bugs of HybridParallelClipGrad

* add unittest of pp with HybridParallelClipGrad

* fix bugs in mp_layers.py

* update

* fix bugs in pp_layers.py

* update

* [HybridParallel]Rebuild code for pipeline (#36396)

* add no_sync for parameters sync

* add pipeline for moe

* [HybridParallel]Support fp16 in dygraph hybrid parallel (#36420)

* [HybridParallel]Support fp16 in dygraph hybrid parallel

* update

* update

* update for recompute

* add unittest of pp+fp16

* add unittest of recompute+fp16

* update

* modify ut

* modify ut of cond (#36475)

* fix bugs of ClipGradByGlobalNorm in HybridParallel (#36555)

* fix bugs of ClipGradByGlobalNorm

* add unittests

* add unittests

* [HybridParallel]fix bug of check_inf in fleet_base.py (#36651)

* fix bug of check_inf

* fix allreduce

* support ClipGradByGlobalNorm in sharding (#36012)

* support ClipGradByGlobalNorm in sharding

* support ClipGradByGlobalNorm in sharding

* test=allcase

* Update test_linalg_cond.py

* Update hybrid_parallel_util.py

* Update hybrid_parallel_util.py

Co-authored-by: ShenLiang <[email protected]>
Co-authored-by: zhaoyingli <[email protected]>
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