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

json parser for annotation parsing #692

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

chadrick-kwag
Copy link

  • added annotation json parser.
  • added --annotationformat option in flow command. If this option is not specified, then it will by default use the PASCAL-VOC-XML parser which is the legacy method.
  • added /docs directory in root and added doc on json format for annotation

for the json format, check out the docs.

@chadrick-kwag
Copy link
Author

chadrick-kwag commented Apr 3, 2018

hmm.. the ci failure is an assertion error while loading the weights...

self = <darkflow.utils.loader.weights_walker object at 0x7fce80cb4dd8>
size = 16
    def walk(self, size):
        if self.eof: return None
        end_point = self.offset + 4 * size
        assert end_point <= self.size, \
>       'Over-read {}'.format(self.path)
E       AssertionError: Over-read /home/travis/build/thtrieu/darkflow/bin/tiny-yolo-voc.weights
darkflow/utils/loader.py:127: AssertionError

and other checks has passed

============== 1 failed, 5 passed, 1 warnings in 78.80 seconds ================

My PR has nothing to do with loading the weights so, can we just ignore this failure?

@abagshaw
Copy link
Collaborator

abagshaw commented Apr 4, 2018

@chadrick-kwag CI failure does seem to be unrelated - probably due to some updated weights or something. I'll check into that when I have a chance.

@monoloxo
Copy link

monoloxo commented Dec 7, 2018

is this PR helpful for training on coco dataset?
why can't i find the code about this in thtrieu/darkflow?
is there some way can training on coco dataset?cause voc is too small sometimes

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