Skip to content

Conversation

@Dayuxiaoshui
Copy link
Contributor

  • Remove temporary workaround code from PR [Bug Fix] Restrict the running_var parameter of BatchNorm to be greater than 0. #301
  • Add min_val constraint handling in replay_tensor function
  • Update convert_meta_classes_to_tensors to read min_val from weight_meta
  • Batch update 801 weight_meta.py files to add min_val=0 for all running_var parameters
  • Fix resolves nan issue in max_diff and mean_diff for 150 samples with batch_norm

Verification: max_diff and mean_diff changed from nan to 0.0, all allclose checks pass

PR Category

Description

@paddle-bot
Copy link

paddle-bot bot commented Oct 25, 2025

Thanks for your contribution!

- Remove temporary workaround code from PR PaddlePaddle#301
- Add min_val constraint handling in replay_tensor function
- Update convert_meta_classes_to_tensors to read min_val from weight_meta
- Batch update 801 weight_meta.py files to add min_val=0 for all running_var parameters
- Fix resolves nan issue in max_diff and mean_diff for 150 samples with batch_norm

Verification: max_diff and mean_diff changed from nan to 0.0, all allclose checks pass
@Dayuxiaoshui
Copy link
Contributor Author

@JewelRoam 感觉很奇怪,为什么会在环境配置出错呢

@Xreki
Copy link
Collaborator

Xreki commented Oct 28, 2025

@JewelRoam 感觉很奇怪,为什么会在环境配置出错呢

@Dayuxiaoshui CI环境搭建问题已经在 #316 中修复了。

Copy link
Collaborator

@Xreki Xreki left a comment

Choose a reason for hiding this comment

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

LGTM

@Dayuxiaoshui
Copy link
Contributor Author

@luotao1 检查完成通过,等待合并

@Xreki Xreki merged commit b4b4623 into PaddlePaddle:develop Oct 29, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants