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

Add Semantic Segmentation of KITTI-STEP Panoptic Segmentation Dataset #1748

Closed
wants to merge 0 commits into from

Conversation

TimoK93
Copy link

@TimoK93 TimoK93 commented Jul 8, 2022

Motivation

In the last years, panoptic segmentation has become more into the focus in reseach. Weber et al. [Link] have published a quite nice dataset, which is in the same style like Cityscapes, but for KITTI sequences. Since Cityscapes and KITTI-STEP share the same classes and also a comparable domain (dashcam view), interesting investigations, e.g. about relations in the domain e.t.c. can be done.

Note that KITTI-STEP provices panoptic segmentation annotations which are out of scope for mmsegmentation.

Modification

Mostly, I added the new dataset and dataset preparation file. To simplify the first usage of the new dataset, I also added configs for the dataset and deeplabv3plus.

BC-breaking (Optional)

No BC-breaking

Use cases (Optional)

Researchers want to test their new methods, e.g. for interpretable AI in the context of semantic segmentation. They want to show, that their method is reproducible on comparable datasets. Thus, they can compare Cityscapes and KITTI-STEP.

@CLAassistant
Copy link

CLAassistant commented Jul 8, 2022

CLA assistant check
All committers have signed the CLA.

@MengzhangLI
Copy link
Contributor

Hi, very thankful for your nice PR. We would review it ASAP.

@codecov
Copy link

codecov bot commented Jul 8, 2022

Codecov Report

Merging #1748 (1ac5cfc) into master (13d4c39) will decrease coverage by 0.22%.
The diff coverage is 36.84%.

❗ Current head 1ac5cfc differs from pull request most recent head b9f52e0. Consider uploading reports for the commit b9f52e0 to get more accurate results

@@            Coverage Diff             @@
##           master    #1748      +/-   ##
==========================================
- Coverage   89.04%   88.81%   -0.23%     
==========================================
  Files         144      145       +1     
  Lines        8636     8674      +38     
  Branches     1458     1463       +5     
==========================================
+ Hits         7690     7704      +14     
- Misses        706      730      +24     
  Partials      240      240              
Flag Coverage Δ
unittests 88.81% <36.84%> (-0.23%) ⬇️

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

Impacted Files Coverage Δ
mmseg/datasets/kitti_step.py 35.13% <35.13%> (ø)
mmseg/datasets/__init__.py 100.00% <100.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 13d4c39...b9f52e0. Read the comment docs.

@@ -0,0 +1,35 @@
_base_ = './kittistep.py'
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, I didn't see any difference between the kittistep.py and kittistep_368x368.py
Would you like to tell us the difference?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes you are right... There is no difference. ...368x368.py could be deleted!


After registration, the data images could be download from [KITTI-STEP](http://www.cvlibs.net/datasets/kitti/eval_step.php)

You may need to follow the following structure for dataset preparation after downloading iSAID dataset.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

iSAID -> KITTI-STEP

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, sorry. Sure!

Comment on lines 40 to 41
img_dir='training_openmmlab/image_02/train',
ann_dir='panoptic_maps_openmmlab/train',
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have a little question, is KITTI-STEP for panoptic segmentation tasks or semantic segmentation? MMSeg only works on semantic segmentation, and panoptic segmentation and instance segmentation are supported in MMDet

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

KITTI-STEP is for panoptic segmentation. But because it follows the Cityscapes label set and the same camera setting (car dashcam view), it is a beautiful dataset to compare sem-seg methods especially if they address ML-interpretatbility. E.g., with this dataset you an compare interclass dependencies or interclass confusion on similar datasets.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see. Thanks for your explanation.
Is there any public research work using it as a sem-seg dataset, since the baseline result in MMSeg needs a public reference?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your effort. The closest baseline can be seen here: http://www.cvlibs.net/publications/Weber2021NEURIPSDATA.pdf in the appendix G. They report STQ metric which is based on SQ(=mIoU). So far, there is no published work using it as a pure Sem-seg dataset. But i am working on it so that i hope it will in some months :) If you think it is more convenient, we also could wait some months and I can create a new PR after publication.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Of course, we really appreciate it that users release their work in mmseg!

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So should i wait with the dataset PR until i used it in a publication?

@TimoK93 TimoK93 closed this Mar 9, 2023
xiexinch added a commit that referenced this pull request May 9, 2023
…#2730)

Note that this PR is a modified version of the withdrawn PR
#1748

## Motivation

In the last years, panoptic segmentation has become more into the focus
in reseach. Weber et al.
[[Link]](http://www.cvlibs.net/publications/Weber2021NEURIPSDATA.pdf)
have published a quite nice dataset, which is in the same style like
Cityscapes, but for KITTI sequences. Since Cityscapes and KITTI-STEP
share the same classes and also a comparable domain (dashcam view),
interesting investigations, e.g. about relations in the domain e.t.c.
can be done.

Note that KITTI-STEP provices panoptic segmentation annotations which
are out of scope for mmsegmentation.

## Modification

Mostly, I added the new dataset and dataset preparation file. To
simplify the first usage of the new dataset, I also added configs for
the dataset, segformer and deeplabv3plus.

## BC-breaking (Optional)

No BC-breaking

## Use cases (Optional)

Researchers want to test their new methods, e.g. for interpretable AI in
the context of semantic segmentation. They want to show, that their
method is reproducible on comparable datasets. Thus, they can compare
Cityscapes and KITTI-STEP.

---------

Co-authored-by: CSH <40987381+csatsurnh@users.noreply.github.com>
Co-authored-by: csatsurnh <cshan1995@126.com>
Co-authored-by: 谢昕辰 <xiexinch@outlook.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.

5 participants