-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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] Remove fp16
folder in configs
.
#1031
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1031 +/- ##
=======================================
Coverage 89.55% 89.55%
=======================================
Files 119 119
Lines 6626 6626
Branches 1034 1034
=======================================
Hits 5934 5934
Misses 488 488
Partials 204 204
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@@ -73,7 +73,6 @@ Supported methods: | |||
- [x] [UNet (MICCAI'2016/Nat. Methods'2019)](configs/unet) | |||
- [x] [PSPNet (CVPR'2017)](configs/pspnet) | |||
- [x] [DeepLabV3 (ArXiv'2017)](configs/deeplabv3) | |||
- [x] [Mixed Precision (FP16) Training (ArXiv'2017)](configs/fp16) | |||
- [x] [BiSeNetV1 (ECCV'2018)](configs/bisenetv1) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we add one item named supported training tricks
in the future?
@@ -129,7 +129,7 @@ Please refer to [CGNet](https://github.com/open-mmlab/mmsegmentation/blob/master | |||
|
|||
### Mixed Precision (FP16) Training | |||
|
|||
Please refer [Mixed Precision (FP16) Training](https://github.com/open-mmlab/mmsegmentation/blob/master/configs/fp16) for details. | |||
Please refer [Mixed Precision (FP16) Training] on BiSeNetV2 (https://github.com/open-mmlab/mmsegmentation/blob/master/configs/bisenetv2/bisenetv2_fcn_fp16_4x4_1024x1024_160k_cityscapes.py) for details. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems that some algorithms are missing in the model zoo, should add in the future.
Please merge the master branch into your branch, thank you. |
…into remove_fp16_folder
* remove fp16 folder * remove fp16 in docs * fix some typos * fix some typos * fix fp16 in yml
* changed training example to add option to train model that predicts x0 (instead of eps), changed DDPM pipeline accordingly * Revert "changed training example to add option to train model that predicts x0 (instead of eps), changed DDPM pipeline accordingly" This reverts commit c5efb525648885f2e7df71f4483a9f248515ad61. * changed training example to add option to train model that predicts x0 (instead of eps), changed DDPM pipeline accordingly * fixed code style Co-authored-by: lukovnikov <lukovnikov@users.noreply.github.com>
Remove 4 FP16 config files into their corresponding folders.