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

Fix test failures due to http rate limit #62

Merged
merged 2 commits into from
Sep 9, 2023
Merged

Conversation

stes
Copy link
Member

@stes stes commented Sep 9, 2023

We observed a test failure in #59 , most likely due to rate limits when downloading raw data for testing from github. This PR replaces some of the links with references to figshare which is probably better suited to host the files.

@cla-bot cla-bot bot added the CLA signed label Sep 9, 2023
@stes
Copy link
Member Author

stes commented Sep 9, 2023

Small typo --- now test_dlc.py works locally for me

19 passed in 36.42s

@MMathisLab MMathisLab merged commit 16653e2 into main Sep 9, 2023
8 checks passed
@MMathisLab MMathisLab deleted the stes/fix-dlc-test branch September 12, 2023 08:25
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.

2 participants