Skip to content
This repository has been archived by the owner on Jan 18, 2024. It is now read-only.

Include support to COCO dataset format #5

Closed
chanfr opened this issue Jul 22, 2017 · 1 comment · Fixed by #38
Closed

Include support to COCO dataset format #5

chanfr opened this issue Jul 22, 2017 · 1 comment · Fixed by #38
Assignees

Comments

@chanfr
Copy link
Member

chanfr commented Jul 22, 2017

No description provided.

@vinay0410
Copy link
Collaborator

vinay0410 commented May 22, 2018

Hi @chanfr ,
This is regarding parsing JSON files using COCO dataset, and since we have agreed on using rapidjson. I would like to mention that rapidjson is already a dependency in DetectionSuite.

So, we can continue to write code using rapidjson, or if neccesary, we can also remove this external dependency.
Since it is header only library, it can be directly used in our code without the need to be installed. This way we can maintain its version compatibility since users might have different versions installed.

I had to update my rapidjson version in ubuntu to fix some errors.

Which way do you think is the best way to go ?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants