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

Rle mask export #4732

Closed
wants to merge 22 commits into from
Closed

Rle mask export #4732

wants to merge 22 commits into from

Conversation

kkm-atom
Copy link

@kkm-atom kkm-atom commented Jul 6, 2022

Create Brush tool by extending this branch and making changes.

Above mentioned branch has been taken for local development. latest "develop" branch is merged to keep everything up to date. Brush tool was already added in the UI but the export to COCO was incorrect. Changes have been made to export it in RLE Mask format.

There's an issue while uploading COCO. The mask does not show up. Same thing is happening in case of importing ellipse via COCO Json. This issue needs resolution

Motivation and context

Linked issues:
#4727
#4723
#1849
#2384

How has this been tested?

Checklist

License

  • I submit my code changes under the same MIT License that covers the project.
    Feel free to contact the maintainers if that's a concern.
  • I have updated the license header for each file (see an example below)
# Copyright (C) 2022 Intel Corporation
#
# SPDX-License-Identifier: MIT

@nmanovic
Copy link
Contributor

nmanovic commented Jul 6, 2022

@kkm-atom , thanks for the contribution! Please give us some time to review and improve the PR. I hope to come back with more details in several weeks.

@nmanovic
Copy link
Contributor

nmanovic commented Aug 2, 2022

@kkm-atom , we will continue the implementation in the repository for now: #2.

@bsekachev
Copy link
Member

Hi, @kkm-atom

Now, I will be able to finish the patch gradually in this repository and maybe to use some of your code if you do not mind.
Looking into a number of conflicts with develop branch and probably with current masks patch (which was adjusted and also has lots of conflicts with develop) it would be easier to close this pull request. Anyway, thank you for the contributing.

@bsekachev bsekachev closed this Aug 21, 2022
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.

3 participants