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

use nested folder #1687

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

waragai-katsunori
Copy link

@waragai-katsunori waragai-katsunori commented Jul 3, 2023

why

  • Curent version assumes images are in non-nested folders train2017 or val2017
  • However nested folder is easier to know what sort of scenes are selected for training and evaluation.

what

  • As far as COCO json file assumes non-nested folders, keep original img_file.
  • If COCO json file assumes nested folders, use img_file for nested case.

Example

In case following folder structure,
keeping data/scene1 as traditional folder structure for COCO Json file format.
I have combined each json files with using nested folder structure
annotations/instances_train2017.json
annotations/instances_val2017.json

Now file_name is such like "file_name": "scene1/filename_1.jpg".

I wish this is useful.

data
data/annotations
data/scene1
data/scene1/annotations
data/scene1/train2017
data/scene1/val2017
data/scene2
data/scene2/annotations
data/scene2/train2017
data/scene2/val2017

@CLAassistant
Copy link

CLAassistant commented Jul 3, 2023

CLA assistant check
All committers have signed the CLA.

@aznhe21 aznhe21 deleted the feature/nested-folder branch March 13, 2024 06:58
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.

2 participants