Skip to content

test: Remove datalad fetch of gin dataset, test against examples - #88

Merged
effigies merged 2 commits into
bids-standard:mainfrom
effigies:test/drop-gin
Aug 31, 2026
Merged

test: Remove datalad fetch of gin dataset, test against examples#88
effigies merged 2 commits into
bids-standard:mainfrom
effigies:test/drop-gin

Conversation

@effigies

Copy link
Copy Markdown
Contributor

GIN is becoming too flaky. We shouldn't have been hitting it up without caching, but the use of datalad was unnecessary, given that we now load the bids-examples datasets. I included eeg_matchingpennies and a few other datasets.

@codecov

codecov Bot commented Aug 31, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.51%. Comparing base (664f751) to head (421e392).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #88      +/-   ##
==========================================
+ Coverage   91.02%   91.51%   +0.49%     
==========================================
  Files          15       15              
  Lines        1136     1084      -52     
  Branches      193      186       -7     
==========================================
- Hits         1034      992      -42     
+ Misses         59       51       -8     
+ Partials       43       41       -2     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

=== Do not change lines below ===
{
 "chain": [],
 "cmd": "uv lock",
 "exit": 0,
 "extra_inputs": [],
 "inputs": [
  "pyproject.toml",
  "uv.lock"
 ],
 "outputs": [
  "uv.lock"
 ],
 "pwd": "."
}
^^^ Do not change lines above ^^^
@effigies
effigies merged commit d343a99 into bids-standard:main Aug 31, 2026
23 checks passed
@effigies
effigies deleted the test/drop-gin branch August 31, 2026 14:59
@yarikoptic

Copy link
Copy Markdown
Contributor

and if ever need any other dataset hosted, let me know and I will do off datasets.datalad.org or alike.

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.

2 participants