Skip to content

Dataloader checking - #75

Open
ChenglongWang wants to merge 6 commits into
devfrom
check_dataloader
Open

Dataloader checking#75
ChenglongWang wants to merge 6 commits into
devfrom
check_dataloader

Conversation

@ChenglongWang

Copy link
Copy Markdown
Contributor
  • Moved datachecking-related functions to check_data_utils.py.
  • Extract check_dataloader function to enable checking dataloader.
  • Minor bugs and typing issues fixed.

Signed-off-by: Chenglong Wang ryuu.j.ching@gmail.com

Extract `check_dataloader` function to enable checking dataloader.
Minor bugs and typing issues fixed.
Signed-off-by: Chenglong Wang <ryuu.j.ching@gmail.com>
@ChenglongWangChenglongWang added the enhancement New feature or request label Nov 24, 2022
@ChenglongWang

ChenglongWang commented Nov 24, 2022

Copy link
Copy Markdown
ContributorAuthor

Potential bugs in check_data_utils.py Line 170 and Line 216 need to be fixed. @ldxFAIRYTAIL
Line 180 caused crash when drawing contour.

Signed-off-by: Chenglong Wang <ryuu.j.ching@gmail.com>
Comment threadstrix/data_checker.py
Comment threadstrix/data_checker.py
)

save_fnames(data, img_key + "_meta_dict", output_fpath)
overlap_m = "mask" if cargs.mask_overlap else "contour" if cargs.contour_overlap else None

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

How about set mask-overlap as the default overlap_m?

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

Labels

enhancementNew feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@ChenglongWang@ldxFAIRYTAIL