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

support for ms coco dataset #13

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

Conversation

raingo
Copy link

@raingo raingo commented May 12, 2015

a working copy of voc_pascal.py for Microsoft COCO dataset.

COCO dataset should be setup by data/coco/PythonAPI/init.py

Trained models will be shared soon.

for row_number in range(len(column)):
box_tmp = reader[column[row_number]][:]
row_data.append(np.transpose(box_tmp))
boxes = row_data

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't this be boxes.append(row_data)?

@raingo
Copy link
Author

raingo commented Jun 21, 2015

@rohitgirdhar It's the results of debugging. I don't know why it should be like that. There are two variables in the mat file. One is a cell array of path strings, and the other is a cell array of matrices (bbox).

@damro
Copy link

damro commented Jun 27, 2015

@raingo Could you please share your trained models for COCO?

@rbgirshick
Copy link
Owner

I've been working on a COCO baseline and have posted the code to the coco branch.

My version of lib/datasets/coco.py is a bit different in that it handles crowd annotations, uses the python coco api for evaluation, and supports proposals in the format generated by Hosang et al..

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.

4 participants