Skip to content

Semantic Segmentation #657

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
wants to merge 6 commits into
base: staging
Choose a base branch
from

Conversation

seanytak
Copy link
Member

@seanytak seanytak commented Sep 28, 2021

Generic Semantic Segmentation Training Tools using PyTorch and Azure Machine Learning

Adds the following to the contrib section

  • Training script for semantic segmentation model using Azure Machine Learning
  • Utilities to convert from various semantic segmentation label formats
  • Utilities to work with semantic segmentation masks
  • PyTorch dataset for semantic segmentation

Related Issues

Checklist:

  • I have followed the contribution guidelines and code style for this project.
  • This branch is created from staging and not master.
  • This PR is being made to staging and not master.
  • I will squash merge this PR into staging.
  • I have added tests covering my contributions.
  • I have updated the documentation accordingly.

@PatrickBue
Copy link
Contributor

Thanks for this contribution. Could you please add some more information what this PR does, how to install, with 1-2 screenshots, etc. For example the segmentation/readme file currently is empty.

@PatrickBue
Copy link
Contributor

@seanytak Feel free to ping me should you have any questions or comments.

@seanytak seanytak force-pushed the setakafu/segmentation branch from 4520c65 to 5798efb Compare October 12, 2021 22:45
@seanytak seanytak marked this pull request as ready for review October 25, 2021 17:53
@seanytak seanytak requested a review from PatrickBue as a code owner October 25, 2021 17:53
@PatrickBue
Copy link
Contributor

@seanytak Great code, all looks good. Let me know once you're ready and I'll merge the code. Two minor comments:

  • could you please update the contrib/README.md file and add your approach to the table.
  • you could add some more info in your own readme.md file, e.g. add screenshots to show what the code does on a high-level. Right now this isn't clear.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants