Skip to content

add multitacred datasets script#32

Open
ArneBinder wants to merge 4 commits intomainfrom
add_multitacred_dataset
Open

add multitacred datasets script#32
ArneBinder wants to merge 4 commits intomainfrom
add_multitacred_dataset

Conversation

@ArneBinder
Copy link
Copy Markdown
Owner

@ArneBinder ArneBinder commented Nov 8, 2023

TODO:

  • add BASE_DATASET_REVISION
  • add tests
  • add requirements.txt
  • add dataset card, i.e. README.md

@codecov
Copy link
Copy Markdown

codecov bot commented Nov 8, 2023

Codecov Report

Attention: 23 lines in your changes are missing coverage. Please review.

Comparison is base (e2803b2) 93.07% compared to head (7121806) 91.70%.

Files Patch % Lines
dataset_builders/pie/multitacred/multitacred.py 64.06% 23 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #32      +/-   ##
==========================================
- Coverage   93.07%   91.70%   -1.38%     
==========================================
  Files          17       18       +1     
  Lines        1286     1350      +64     
==========================================
+ Hits         1197     1238      +41     
- Misses         89      112      +23     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ArneBinder ArneBinder force-pushed the add_multitacred_dataset branch from 8b371cd to eb8d374 Compare November 16, 2023 09:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant