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: fix the config of squeezenet_1.0 with amp_level O0 #769

Merged
merged 1 commit into from
Mar 19, 2024

Conversation

PingqiLi
Copy link
Contributor

@PingqiLi PingqiLi commented Mar 19, 2024

Thank you for your contribution to the MindCV repo.
Before submitting this PR, please make sure:

Motivation

Adjust the amp_level from O2 to O0 to avoid overflow happened during the training of squeezenet_1.0.

Test Plan

(How should this PR be tested? Do you require special setup to run the test or repro the fixed bug?)

Related Issues and PRs

(Is this PR part of a group of changes? Link the other relevant PRs and Issues here. Use https://help.github.com/en/articles/closing-issues-using-keywords for help on GitHub syntax)

@PingqiLi PingqiLi changed the title fix the config of squeezenet_1.0 with amp_level O0 fix: fix the config of squeezenet_1.0 with amp_level O0 Mar 19, 2024
@geniuspatrick geniuspatrick merged commit bd5071f into mindspore-lab:main Mar 19, 2024
5 checks passed
Ash-Lee233 added a commit that referenced this pull request Mar 19, 2024
* fix(workflow): migrate pypi authentication to api tokens (#757)

* fix deeplabv3 (#760)

* fix: Use custom groupconv in shufflenet_v1 instead of nn.Conv2d (#763)

* fix pvt loss issue via SATURATION_MODE (#764)

* fix: add warmup to the training config of squeezenet_1.0 to make the training stable. (#762)

* fix: fix bugs of pvt (#766)

* fix: fix the bug of ResizeBilinear for pvt (#767)

* vit_update: initialize method and do pre-scale before softmax; update configuration yaml and ckpt files (#765)

* fix the config of squeezenet_1.0 with amp_level O0 (#769)

---------

Co-authored-by: Genius Patrick <74176172+geniuspatrick@users.noreply.github.com>
Co-authored-by: zhaoting <37293445+CaitinZhao@users.noreply.github.com>
Co-authored-by: The-truthh <53842165+The-truthh@users.noreply.github.com>
Co-authored-by: Pingqi Li <58093835+PingqiLi@users.noreply.github.com>
Co-authored-by: EmmaHAN <107303365+sageyou@users.noreply.github.com>
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.

3 participants