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

[Refactor]: Unified parameter initialization #567

Merged
merged 2 commits into from
Jun 17, 2021

Conversation

xvjiarui
Copy link
Collaborator

No description provided.

@codecov
Copy link

codecov bot commented May 21, 2021

Codecov Report

Merging #567 (0e2b34b) into master (524425c) will decrease coverage by 0.30%.
The diff coverage is 68.05%.

❗ Current head 0e2b34b differs from pull request most recent head fb3391c. Consider uploading reports for the commit fb3391c to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##           master     #567      +/-   ##
==========================================
- Coverage   86.41%   86.11%   -0.31%     
==========================================
  Files         101      101              
  Lines        5279     5235      -44     
  Branches      851      825      -26     
==========================================
- Hits         4562     4508      -54     
  Misses        557      557              
- Partials      160      170      +10     
Flag Coverage Δ
unittests 86.11% <68.05%> (-0.31%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
mmseg/models/segmentors/cascade_encoder_decoder.py 100.00% <ø> (ø)
mmseg/models/segmentors/encoder_decoder.py 88.35% <25.00%> (-2.50%) ⬇️
mmseg/models/backbones/hrnet.py 82.45% <52.94%> (-1.85%) ⬇️
mmseg/models/backbones/cgnet.py 94.63% <53.84%> (-2.13%) ⬇️
mmseg/models/backbones/unet.py 94.87% <53.84%> (-0.93%) ⬇️
mmseg/models/backbones/mobilenet_v2.py 71.08% <57.14%> (-1.86%) ⬇️
mmseg/models/backbones/mobilenet_v3.py 94.82% <57.14%> (-2.66%) ⬇️
mmseg/models/backbones/fast_scnn.py 97.02% <83.33%> (-1.07%) ⬇️
mmseg/models/backbones/resnet.py 99.28% <92.00%> (+1.04%) ⬆️
mmseg/models/backbones/vit.py 86.15% <100.00%> (-0.90%) ⬇️
... and 6 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 524425c...fb3391c. Read the comment docs.

@xvjiarui xvjiarui merged commit 0c5b026 into open-mmlab:master Jun 17, 2021
bowenroom pushed a commit to bowenroom/mmsegmentation that referenced this pull request Feb 25, 2022
* [Refactor]: Unified parameter initialization

* fixed pretrained
aravind-h-v pushed a commit to aravind-h-v/mmsegmentation that referenced this pull request Mar 27, 2023
* Fix typo in docstring.

* Allow dtype to be overridden on model load.

This may be a temporary solution until open-mmlab#567 is addressed.

* Create latents in float32

The denoising loop always computes the next step in float32, so this
would fail when using `bfloat16`.
aravind-h-v pushed a commit to aravind-h-v/mmsegmentation that referenced this pull request Mar 27, 2023
* WIP: flax FlaxDiffusionPipeline & FlaxStableDiffusionPipeline

* todo comment

* Fix imports

* Fix imports

* add dummies

* Fix empty init

* make pipeline work

* up

* Allow dtype to be overridden on model load.

This may be a temporary solution until open-mmlab#567 is addressed.

* Convert params to bfloat16 or fp16 after loading.

This deals with the weights, not the model.

* Use Flax schedulers (typing, docstring)

* PNDM: replace control flow with jax functions.

Otherwise jitting/parallelization don't work properly as they don't know
how to deal with traced objects.

I temporarily removed `step_prk`.

* Pass latents shape to scheduler set_timesteps()

PNDMScheduler uses it to reserve space, other schedulers will just
ignore it.

* Wrap model imports inside availability checks.

* Optionally return state in from_config.

Useful for Flax schedulers.

* Do not convert model weights to dtype.

* Re-enable PRK steps with functional implementation.

Values returned still not verified for correctness.

* Remove left over has_state var.

* make style

* Apply suggestion list -> tuple

Co-authored-by: Suraj Patil <surajp815@gmail.com>

* Apply suggestion list -> tuple

Co-authored-by: Suraj Patil <surajp815@gmail.com>

* Remove unused comments.

* Use zeros instead of empty.

Co-authored-by: Mishig Davaadorj <dmishig@gmail.com>
Co-authored-by: Mishig Davaadorj <mishig.davaadorj@coloradocollege.edu>
Co-authored-by: Patrick von Platen <patrick.v.platen@gmail.com>
Co-authored-by: Suraj Patil <surajp815@gmail.com>
sibozhang pushed a commit to sibozhang/mmsegmentation that referenced this pull request Mar 22, 2024
* resolve comments

* update changelog

* Refactor AVA Eval, only support mAP

* fix unittest

* add warning

* Refactor AVA Eval, only support mAP

* fix unittest

* add warning

* update warning info
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.

1 participant