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

upload datasets via GitHub CI workflow [WIP] #27

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ArneBinder
Copy link
Owner

@ArneBinder ArneBinder commented Nov 7, 2023

add code, requirements.txt and github workflow from pytorch-ie

TODO:

  • update code to work with current package versions
  • test it

Copy link

codecov bot commented Nov 7, 2023

Codecov Report

Merging #27 (db0b3f8) into main (1e04493) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #27   +/-   ##
=======================================
  Coverage   92.77%   92.77%           
=======================================
  Files          13       13           
  Lines        1148     1148           
=======================================
  Hits         1065     1065           
  Misses         83       83           

📣 Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!

ArneBinder added a commit to ArneBinder/pytorch-ie that referenced this pull request Nov 8, 2023
* create document fixture without any dataset class

* remove any dataset related classes; remove statistic implementations

* remove PIE dataset loading scripts (they will be added to pie-datasets, see ArneBinder/pie-datasets#22)

* remove dataset specific nox test sessions

* cleanup

* add pandas dependency because it is required in metrics (f1 and statistics)

* fix nox mypy session

* export RequiresDocumentTypeMixin and DocumentStatistic in core

* remove workflow to upload datasets to the Huggingface hub (in favor of ArneBinder/pie-datasets#27)
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.

1 participant