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

Jump to bottom

Add data_loader sub-package #35

Open

aphearin wants to merge 1 commit into main from data_loader

Collaborator

aphearin commented Dec 16, 2024

@alexalar this PR brings in a new data_loaders sub-package with the basic features we discussed today during group meeting. A few things to note:

I modified diffstarpop/.coveragerc so that code inside data_loaders will not count towards the coverage fraction
There is one unit test that enforces that dummy_data_loader imports. This unit test will run in our CI.
There is a separate unit test that will only run on a machine where the data exists.


          Add data_loader sub-package

4fd65b5

# for free to join this conversation on GitHub. Already have an account? # to comment

Labels

None yet