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

[ENH] Generate BIDS Derivatives-compatible outputs #46

Merged
merged 36 commits into from
Nov 22, 2021
Merged
Changes from 1 commit
Commits
Show all changes
36 commits
Select commit Hold shift + click to select a range
d0207cf
Start drafting BIDS-compatible outputs.
tsalo Feb 13, 2021
b5f100a
Fix bug.
tsalo Sep 15, 2021
196b4be
Make dictionaries optional.
tsalo Sep 15, 2021
8c20cf2
Test the dictionaries.
tsalo Sep 15, 2021
c2c0fbf
Of course it'll evaluate an empty dict to False.
tsalo Sep 15, 2021
2081d86
Merge branch 'main' into derivatives
tsalo Sep 15, 2021
8f4d075
Fix style issues.
tsalo Sep 15, 2021
4aab865
Fix everything.
tsalo Sep 15, 2021
bedb83e
Try again...
tsalo Sep 15, 2021
c9f287c
Update test_integration.py
tsalo Sep 15, 2021
805dae6
Update test_integration.py
tsalo Sep 15, 2021
9dc9588
Update utils.py
tsalo Sep 15, 2021
fc685a3
Simplify classification, hopefully.
tsalo Sep 15, 2021
e526458
Merge branch 'main' into derivatives
tsalo Sep 29, 2021
f022606
No mask.
tsalo Sep 29, 2021
e4cd759
Update test_integration.py
tsalo Sep 29, 2021
6cb64eb
Change remaining files.
tsalo Sep 29, 2021
cae8c23
Reorg test.
tsalo Sep 29, 2021
b959e66
Update utils.py
tsalo Sep 29, 2021
fd0327f
More stuff.
tsalo Sep 29, 2021
c09adb9
*sigh*
tsalo Sep 29, 2021
a6c4567
Update test_integration.py
tsalo Sep 29, 2021
b6a1ca0
Update utils.py
tsalo Sep 29, 2021
ebc2f9d
Update test_integration.py
tsalo Sep 29, 2021
a704768
Revert "Update test_integration.py"
tsalo Sep 29, 2021
33fb437
Update test_integration.py
tsalo Sep 29, 2021
f186a16
Split up classification and filewriting.
tsalo Sep 30, 2021
a85824f
Some cleanup.
tsalo Sep 30, 2021
c351f5a
Update test_integration.py
tsalo Sep 29, 2021
0ae13cb
Revert "Update test_integration.py"
tsalo Sep 29, 2021
47efc5f
Merge branch 'main' of https://github.com/tsalo/aroma into main
tsalo Sep 30, 2021
113d956
Merge branch 'main' into derivatives
tsalo Sep 30, 2021
df3e8d6
Merge branch 'main' into derivatives
tsalo Nov 19, 2021
16e555d
Add seed
eurunuela Nov 22, 2021
cae4937
Merge branch 'derivatives' of https://github.com/tsalo/aroma into pr/46
eurunuela Nov 22, 2021
2ea78d2
Update benchmark files
eurunuela Nov 22, 2021
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Update test_integration.py
tsalo committed Sep 29, 2021
commit ebc2f9d5cf7af312c6d0e4c294bc2c516221e491
3 changes: 2 additions & 1 deletion aroma/tests/test_integration.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
from os.path import isfile, join, split
from os.path import abspath, isfile, join, split

import numpy as np
import pandas as pd
@@ -47,6 +47,7 @@ def test_integration(skip_integration, nilearn_data):
assert isfile(join(out_path, "classified_motion_ICs.txt"))
assert isfile(join(out_path, "denoised_func_data_nonaggr.nii.gz"))
assert isfile(join(out_path, "feature_scores.tsv"))
raise Exception(abspath(out_path))

# Check classification overview file
true_classification_overview = pd.read_csv(join(resources_path, "classification_overview.txt"),