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] Support format_result and fix prefix param in cityscape metric, and rename CitysMetric to CityscapesMetric #2660

Merged
merged 3 commits into from
Mar 7, 2023

Conversation

MeowZheng
Copy link
Collaborator

@MeowZheng MeowZheng commented Feb 28, 2023

as title

@codecov
Copy link

codecov bot commented Feb 28, 2023

Codecov Report

Patch coverage: 85.71% and project coverage change: -0.11 ⚠️

Comparison is base (19f9285) 83.52% compared to head (fd22d79) 83.42%.

❗ Current head fd22d79 differs from pull request most recent head 7c628e6. Consider uploading reports for the commit 7c628e6 to get more accurate results

Additional details and impacted files
@@             Coverage Diff             @@
##           dev-1.x    #2660      +/-   ##
===========================================
- Coverage    83.52%   83.42%   -0.11%     
===========================================
  Files          147      147              
  Lines         8621     8626       +5     
  Branches      1293     1295       +2     
===========================================
- Hits          7201     7196       -5     
- Misses        1195     1207      +12     
+ Partials       225      223       -2     
Flag Coverage Δ
unittests 83.42% <85.71%> (-0.11%) ⬇️

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

Impacted Files Coverage Δ
mmseg/evaluation/metrics/citys_metric.py 86.66% <84.21%> (-3.34%) ⬇️
mmseg/evaluation/__init__.py 100.00% <100.00%> (ø)
mmseg/evaluation/metrics/__init__.py 100.00% <100.00%> (ø)
mmseg/datasets/transforms/transforms.py 89.49% <0.00%> (-0.91%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@MeowZheng MeowZheng changed the title [WIP] Fix prefix parameter in citys metric [Fix] Fix prefix parameter in citys metric Mar 1, 2023
@MeowZheng MeowZheng changed the title [Fix] Fix prefix parameter in citys metric [Fix] Support format_result and fix prefix param in cityscape metric Mar 6, 2023
@MeowZheng MeowZheng changed the title [Fix] Support format_result and fix prefix param in cityscape metric [Fix] Support format_result and fix prefix param in cityscape metric, and rename CitysMetric to CityscapesMetric Mar 6, 2023
@MeowZheng MeowZheng merged commit a8aafdd into open-mmlab:dev-1.x Mar 7, 2023
aravind-h-v pushed a commit to aravind-h-v/mmsegmentation that referenced this pull request Mar 27, 2023
nahidnazifi87 pushed a commit to nahidnazifi87/mmsegmentation_playground that referenced this pull request Apr 5, 2024
… and rename CitysMetric to CityscapesMetric (open-mmlab#2660)

as title
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.

None yet

3 participants