Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[Features] Intialize dataset with ann_file #122

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

Conversation

MeowZheng
Copy link
Collaborator

Motivation

  1. As title

Modification

  1. Add ann_file and file_client_args in BaseDataset
  2. Add load_ann_file in BaseDataset

@codecov
Copy link

codecov bot commented Apr 23, 2022

Codecov Report

Merging #122 (0fe8eac) into master (579b68b) will increase coverage by 0.07%.
The diff coverage is 31.25%.

@@            Coverage Diff             @@
##           master     #122      +/-   ##
==========================================
+ Coverage   68.78%   68.85%   +0.07%     
==========================================
  Files          92       92              
  Lines        5135     5150      +15     
  Branches      727      729       +2     
==========================================
+ Hits         3532     3546      +14     
  Misses       1489     1489              
- Partials      114      115       +1     
Flag Coverage Δ
unittests 68.85% <31.25%> (+0.07%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
mmflow/datasets/base_dataset.py 80.00% <31.25%> (-13.34%) ⬇️
mmflow/datasets/pipelines/transforms.py 79.16% <0.00%> (+1.66%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 579b68b...0fe8eac. Read the comment docs.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant