Skip to content

[Feature] Add opacity option to show_result #425

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Mar 23, 2021

Conversation

daavoo
Copy link
Contributor

@daavoo daavoo commented Mar 15, 2021

Add new optional argument to show_result for changing the opacity of the painted segmentation map. The default value maintains the previous behavior. Places using show_result had been updated exposing the new argument.

@codecov
Copy link

codecov bot commented Mar 15, 2021

Codecov Report

Merging #425 (1128225) into master (7e1b24d) will decrease coverage by 0.01%.
The diff coverage is 25.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #425      +/-   ##
==========================================
- Coverage   86.44%   86.42%   -0.02%     
==========================================
  Files          97       97              
  Lines        4958     4959       +1     
  Branches      805      805              
==========================================
  Hits         4286     4286              
- Misses        517      518       +1     
  Partials      155      155              
Flag Coverage Δ
unittests 86.42% <25.00%> (-0.02%) ⬇️

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

Impacted Files Coverage Δ
mmseg/apis/test.py 22.68% <ø> (ø)
mmseg/models/segmentors/base.py 44.95% <0.00%> (-0.42%) ⬇️
mmseg/apis/inference.py 66.07% <50.00%> (ø)

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 7e1b24d...5b06325. Read the comment docs.

@xvjiarui xvjiarui requested a review from xiexinch March 17, 2021 02:29
@xvjiarui xvjiarui merged commit 6acd77e into open-mmlab:master Mar 23, 2021
bowenroom pushed a commit to bowenroom/mmsegmentation that referenced this pull request Feb 25, 2022
aravind-h-v pushed a commit to aravind-h-v/mmsegmentation that referenced this pull request Mar 27, 2023
* [MPS] Make sure it doesn't break torch < 1.12

* up
wjkim81 pushed a commit to wjkim81/mmsegmentation that referenced this pull request Dec 3, 2023
Signed-off-by: lizz <lizz@sensetime.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants