[Feature] Support DSDL Dataset #2925
Conversation
wufan-tb
commented
Apr 19, 2023
wufan-tb
commented
- support dsdl seg dataset
- add dsdl dataset citest
- validated accuracy on voc2012 and cityscapes
|
|
configs/dsdl/cityscapes.py
Outdated
| train_ann = 'dsdl/set-train/train.yaml' | ||
| val_ann = 'dsdl/set-val/val.yaml' |
There was a problem hiding this comment.
Should we create the dsdl folder at mmsegmentation root path or at the datasets root path?
There was a problem hiding this comment.
this path will concate with root_path, so the actually path is "data/cityscapes/dsdl/set-train/train.yaml".
furthermore, here is a typical example of dsdl dataset's file struct :
data
└── VOC12-seg
├── dsdl
└── original
There was a problem hiding this comment.
Since users may not know how to use this dataset interface, could you write a document to introduce how to generate the train.yaml and val.yaml, or provide an example?
configs/dsdl/voc.py
Outdated
| # dataset settings | ||
| dataset_type = 'DSDLSegDataset' | ||
| data_root = 'data/VOC12-seg' | ||
| img_prefix = 'original' |
There was a problem hiding this comment.
Is 'original' a keyword needed inside DSDL?
There was a problem hiding this comment.
no, users should change it by actual path. (or no img_prefix, decied by it's file sturct)
There was a problem hiding this comment.
It is recommended not to use original and try to be consistent with https://github.com/open-mmlab/mmsegmentation/blob/main/configs/_base_/datasets/pascal_voc12_aug.py#L47, since other users may not notice this detail.
configs/dsdl/cityscapes.py
Outdated
| train_ann = 'dsdl/set-train/train.yaml' | ||
| val_ann = 'dsdl/set-val/val.yaml' |
There was a problem hiding this comment.
Since users may not know how to use this dataset interface, could you write a document to introduce how to generate the train.yaml and val.yaml, or provide an example?
configs/dsdl/voc.py
Outdated
| # dataset settings | ||
| dataset_type = 'DSDLSegDataset' | ||
| data_root = 'data/VOC12-seg' | ||
| img_prefix = 'original' |
There was a problem hiding this comment.
It is recommended not to use original and try to be consistent with https://github.com/open-mmlab/mmsegmentation/blob/main/configs/_base_/datasets/pascal_voc12_aug.py#L47, since other users may not notice this detail.
wufan-tb
force-pushed
the
dev-1.x-dsdl
branch
4 times, most recently
from
9d7876c
to
8920dd4
Compare
- support dsdl seg dataset - add dsdl dataset citest - validated accuracy on voc2012 and cityscapes
