Skip to content
This repository has been archived by the owner on Oct 31, 2023. It is now read-only.

Panoptic Segmentation #928

Merged
merged 2 commits into from
Jun 25, 2019
Merged

Panoptic Segmentation #928

merged 2 commits into from
Jun 25, 2019

Conversation

karanchahal
Copy link
Contributor

Hello,

This pull request adds a demo which modifies a Mask RCNN to construct a Panoptic Feature Pyramid Network. The google colab notebook linke is given here. The notebook details an example which trains a model on the shapes dataset that outputs:

  1. Bounding boxes and instance segmentation masks via the Mask RCNN
  2. Semantic segmentation masks through the Panoptic Segmentation Head which differentiates the background and foreground.

I hope to build on this demo and add full fledged Panoptic Segmentation support ( #337 ) to the repo, this demo is meant to be as a proof of concept (or a sanity check) that things work.

The basic stuff is here, however we will need to add support for Panoptic Evaluation and modify the structures directory to support the panoptic format. I would love to discuss on what would be the best way to add this support and perform ablation studies on verifying the accuracy of the model.

Would love to know what you think @fmassa !

Thanks,
Karan

@facebook-github-bot facebook-github-bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Jun 25, 2019
@fmassa
Copy link
Contributor

fmassa commented Jun 25, 2019

I think this is a great idea!

I'm glad to merge this as is.

Also, it might be good to connect with the authors of https://arxiv.org/abs/1906.05896
They have extended maskrcnn-benchmark to support Panoptic Segmentation, I was wondering if they were planning on open-sourcing their implementation?

cc @jlazarow , the main author of https://arxiv.org/abs/1906.05896

@karanchahal
Copy link
Contributor Author

@jlazarow Your thoughts would be much appreciated. What do you think would be the best way to go about including support for Panoptic Segmentation in this repo ?

@fmassa fmassa merged commit 55796a0 into facebookresearch:master Jun 25, 2019
@fmassa
Copy link
Contributor

fmassa commented Jun 25, 2019

Merging this in the meantime, as it's self-contained and looks great!

@jlazarow
Copy link

Hi,

Yes, we will open source the implementation in the near future. I can write some notes/thoughts this week and can upload our rough implementations of the Panoptic FPN/fusion code. I do not expect these to be immediately merge-able (I might have done narrow-minded things in coco_eval.py), but I would be happy to help get it into this repo (I guess before Detectron2 drops).

@karanchahal
Copy link
Contributor Author

Yeah, that would be great ! I'm looking forward to it.

On a side note, what's detectron 2 going to support ? Sounds very interesting.

Ricardozzf added a commit to Ricardozzf/maskrcnn-benchmark that referenced this pull request Jul 4, 2019
@karanchahal
Copy link
Contributor Author

Hello @jlazarow !
Is there any update to this ? I would love to get started on this as per your instructions :D

Lyears pushed a commit to Lyears/maskrcnn-benchmark that referenced this pull request Jun 28, 2020
* Added panoptic segmentation demo

* Removed overlong notebook outputs
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CLA Signed Do not delete this pull request or issue due to inactivity.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants