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

MNT: Pybids 0.7 compatibility #84

Merged
merged 40 commits into from
Dec 12, 2018
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
40 commits
Select commit Hold shift + click to select a range
47fc3fc
PIN: Nistats + pybids
effigies Nov 12, 2018
63ef920
FIX: type->suffix, model.json->smdl.json
effigies Nov 12, 2018
a8ba7b9
FIX: BIDSLayout, derivatives syntax
effigies Nov 12, 2018
c54b075
RF: Update paradigm/confounds to sparse/dense
effigies Nov 26, 2018
0f7ac98
RF: filename->path and other renaming updates
effigies Nov 30, 2018
45180ba
Extract model paths from BIDSFile objects
adelavega Nov 30, 2018
487b62a
Change preproc_dir to more generic derivatives which complies with BI…
adelavega Nov 30, 2018
cb17b01
Fix derivatives typos
adelavega Nov 30, 2018
8079d95
Store derivatives into a single list
effigies Dec 3, 2018
39f912f
Merge pull request #1 from adelavega/fix/modelstring
effigies Dec 3, 2018
377e567
Emulate previous contrast behavior
adelavega Dec 3, 2018
ab4f83a
Begin converting higher level contrasts
adelavega Dec 3, 2018
414f613
Attempt to simplify contrasts
adelavega Dec 5, 2018
93c7dc1
Remove indices/matrix from higher level
adelavega Dec 6, 2018
8a61ee7
Add dataset description to reportlet
adelavega Dec 6, 2018
6f36d5a
TMP: Do not produce correlation matrix plot
effigies Dec 6, 2018
e131491
Merge remove get_ev
adelavega Dec 6, 2018
9b58cb6
PIN: pybids, nipype, nistats
effigies Dec 7, 2018
16ff0fe
Set bids config for custom fitlins entities, and disable validation f…
adelavega Dec 7, 2018
132abf5
Select only bold files
adelavega Dec 7, 2018
b52675e
Fix l1_model input types, and except sparse and dense inputs
adelavega Dec 7, 2018
64c5fb9
Merge branch 'mnt/dependency_updates' into mnt/pybids
effigies Dec 8, 2018
1bb2617
FIX: Update to recent nistats syntax, always set sparse/dense
effigies Dec 8, 2018
a5cafac
FIX: Mask file selection
effigies Dec 8, 2018
65a35ce
Merge branch 'mnt/dependency_updates' of https://github.com/effigies/…
adelavega Dec 9, 2018
e18d3f6
Merge branch 'mnt/dependency_updates' into mnt/pybids
adelavega Dec 9, 2018
deeb3cc
Merge branch 'mnt/pybids' of github.com:adelavega/fitlins into mnt/py…
adelavega Dec 9, 2018
a6f9f8c
Create new contrast dictionary in prepare_contrasts
adelavega Dec 9, 2018
7b87858
FIX: Final cleanups before merge
effigies Dec 10, 2018
a1405c5
Merge pull request #2 from adelavega/mnt/pybids
effigies Dec 10, 2018
96c5ac3
Update add_config_paths syntax
adelavega Dec 10, 2018
d0740ac
Remove capitalizaton of F contrast-- pybids returns correct case
adelavega Dec 10, 2018
528c6ef
Contrast metadata passes name, contrast_info unnest
adelavega Dec 10, 2018
decbdb1
Refactor prepare_contrasts, and second level model
adelavega Dec 12, 2018
e11a38f
Second level running
adelavega Dec 12, 2018
b4c877f
Minor clear up
adelavega Dec 12, 2018
6f5add2
Merge pull request #3 from adelavega/mnt/pybids
effigies Dec 12, 2018
4e0d0de
PIN: Update pybids, nipype pins
effigies Dec 12, 2018
27226a1
STY: Minor cleanups
effigies Dec 12, 2018
1f11b09
ENH: Disable multiple models for now; STY: unused import
effigies Dec 12, 2018
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
9 changes: 4 additions & 5 deletions fitlins/__about__.py
Original file line number Diff line number Diff line change
Expand Up @@ -34,21 +34,20 @@

REQUIRES = [
'nibabel>=2.0',
'nipype>=1.1.2',
'nipype>=1.1.6',
'seaborn>=0.7.1',
'numpy>=1.11',
'nilearn>=0.4.0',
'pandas>=0.19',
'tables>=3.2.1',
'nistats>=0.0.1a',
'grabbit>=0.2.3',
'nistats>=0.0.1b0',
'pybids>=0.6.5',
'jinja2',
]

LINKS_REQUIRES = [
'git+https://github.com/nistats/nistats.git@'
'ce3695e8f34c6f34323766dc96a60a53b69d2729#egg=nistats',
'git+https://github.com/bids-standard/pybids.git@'
'dd8c9bc6478bb63e09f6c95566a11677ce12ded7#egg=pybids',
]

TESTS_REQUIRES = [
Expand Down
16 changes: 5 additions & 11 deletions fitlins/cli/run.py
Original file line number Diff line number Diff line change
Expand Up @@ -79,9 +79,9 @@ def get_parser():
'removed)')
g_bids.add_argument('-m', '--model', action='store', default='model.json',
help='location of BIDS model description (default bids_dir/model.json)')
g_bids.add_argument('-p', '--preproc-dir', action='store', default='fmriprep',
help='location of preprocessed data (if relative path, search '
'bids_dir/derivatives, followed by output_dir)')
g_bids.add_argument('-d', '--derivatives', action='store', nargs='+',
help='location of derivatives (including preprocessed images).'
'If none specified, indexes all derivatives under bids_dir/derivatives.')
g_bids.add_argument('--derivative-label', action='store', type=str,
help='execution label to append to derivative directory name')
g_bids.add_argument('--space', action='store',
Expand Down Expand Up @@ -139,13 +139,7 @@ def run_fitlins(argv=None):
if opts.model in (None, 'default') and not op.exists(model):
model = 'default'

preproc_dir = default_path(opts.preproc_dir,
op.join(opts.bids_dir, 'derivatives'),
'fmriprep')
if not op.exists(preproc_dir):
preproc_dir = default_path(opts.preproc_dir, opts.output_dir, 'fmriprep')
if not op.exists(preproc_dir):
raise RuntimeError("Preprocessed data could not be found")
derivatives = True if not opts.derivatives else opts.derivatives

pipeline_name = 'fitlins'
if opts.derivative_label:
Expand All @@ -158,7 +152,7 @@ def run_fitlins(argv=None):
work_dir = mkdtemp() if opts.work_dir is None else opts.work_dir

fitlins_wf = init_fitlins_wf(
opts.bids_dir, preproc_dir, deriv_dir, opts.space, model=model,
opts.bids_dir, derivatives, deriv_dir, opts.space, model=model,
participants=subject_list, base_dir=work_dir,
include_pattern=opts.include, exclude_pattern=opts.exclude
)
Expand Down
Loading