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.

Sign up for GitHub

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

Jump to bottom

Add image export option #327

Closed
wants to merge 2 commits into from
Closed

Add image export option #327

wants to merge 2 commits into from

Conversation

2knal
Copy link
Contributor

@2knal 2knal commented Mar 7, 2020

Description

Added a button, with which we can create annotation files for all uploaded images.
These annotations also get downloaded as annotations.zip.
The downloaded annotations can further be used for any custom object detection models.

Dependency change:

  • archiverjs
  • xmlbuilderjs

Closes #298

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

On local forked repository.
image
image
image
image

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

@Tushar19varshney
Copy link
Contributor

Rebase the PR!

@2knal 2knal force-pushed the 298-2knal branch 2 times, most recently from 7ef5052 to b3cf801 Compare March 9, 2020 16:35
@2knal
Copy link
Contributor Author

2knal commented Mar 9, 2020 •
edited
Loading

Rebase the PR!

Sorry @Tushar19varshney ,
I have made the necessary changes, please have a look.

@Tushar19varshney
Copy link
Contributor

@2knal Could you please rebase it again ? Also, check the codacy errors as it was failing.

@Tushar19varshney Tushar19varshney added low priority web Issue and PR regarding web app labels May 16, 2020
Add image export option
6004c15 
Setup frontend with actions and reducers.
Receiving request in the backend.
Can create annotations for image files.
Ability to download zipped annotations.
@2knal
Copy link
Contributor Author

2knal commented May 20, 2020

@2knal Could you please rebase it again ? Also, check the codacy errors as it was failing.

Is it alright now?
Sorry for the delayed response.

@ThatUdeshUK
Copy link
Collaborator

Close because of the inactivity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Tushar19varshney Tushar19varshney Awaiting requested review from Tushar19varshney

Assignees
No one assigned
Labels
low priority web Issue and PR regarding web app
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Image export option
3 participants
@2knal @Tushar19varshney @ThatUdeshUK