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

Why instance annotations are required along panoptic ones? #8

Open
aviadmx opened this issue May 16, 2022 · 4 comments
Open

Why instance annotations are required along panoptic ones? #8

aviadmx opened this issue May 16, 2022 · 4 comments

Comments

@aviadmx
Copy link

aviadmx commented May 16, 2022

The model solves the panoptic segmentation task, why does the validation dataset uses the instance segmentation annotations?

data = dict(
    samples_per_gpu=2,
    workers_per_gpu=2,
    train=dict(
        type=dataset_type,
        ann_file= './datasets/annotations/panoptic_train2017_detection_format.json',
        img_prefix=data_root + 'train2017/',
        pipeline=train_pipeline),
    val=dict( 
      
        segmentations_folder='./seg',
        gt_json = './datasets/annotations/panoptic_val2017.json',
        gt_folder = './datasets/annotations/panoptic_val2017',
        type=dataset_type,
        ann_file=data_root + 'annotations/instances_val2017.json', # Why?
        img_prefix=data_root + 'val2017/',
        pipeline=test_pipeline),
    test=dict(
        segmentations_folder='./seg',
        gt_json = './datasets/annotations/panoptic_val2017.json',
        gt_folder = './datasets/annotations/panoptic_val2017',
        type=dataset_type,
        #ann_file= './datasets/coco/annotations/image_info_test-dev2017.json',
        ann_file=data_root + 'annotations/instances_val2017.json', # Why?
        #img_prefix=data_root + '/test2017/',
        img_prefix=data_root + 'val2017/',
        pipeline=test_pipeline)
        )

We eventually use the instances_val2017.json file instead of panoptic_val2017.json

@zhiqi-li
Copy link
Owner

"instances_val2017.json" is used for instance segmentation, the GT of panoptic segmentation is './datasets/annotations/panoptic_val2017.json',

@aviadmx
Copy link
Author

aviadmx commented May 16, 2022

I see, but I'm asking why do we need to supply those and not just the annotation file of panoptic_val2017.json? Isn't it a bit redundant since we're doing panoptic segmentation here?

@zhiqi-li
Copy link
Owner

Because while we perform the panoptic segmentation task, we obtain the instance segmentation results by the way. So that I evaluate the performance of the instance segmentation for a more reference for model performance.

@pratik9989
Copy link

Hello @zhiqi-li @aviadmx

I am getting the below error while starting running training command:
FileNotFoundError: CocoDataset_panoptic: [Errno 2] No such file or directory: 'datasets/coco/annotations/instances_val2017.json'

I checked README.md file and I can't see anything about the instances_val2017.json file.
Do you have any idea about it?

Thanks

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

No branches or pull requests

3 participants