Skip to content

FIX: Add dots to extensions #548

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

Merged
merged 1 commit into from
Aug 12, 2020
Merged

FIX: Add dots to extensions #548

merged 1 commit into from
Aug 12, 2020

Conversation

effigies
Copy link
Member

PyBIDS 0.11.1 has stopped doing extension magic for custom layout configurations (bids-standard/pybids#636), as one assumes that somebody providing custom configs can use them correctly.

This brings niworkflows up-to-date. We were failing to find the anatomical conformation snippet due to the mismatch between the extension entity in nipreps.json and its value in the reporting yml file. For some reason I haven't quite fathomed, it hasn't shown up on Circle until yesterday, but now it's reliable.

@pull-assistant
Copy link

pull-assistant bot commented Aug 11, 2020

Score: 1.00

Best reviewed: commit by commit


Optimal code review plan

     FIX: Add dots to extensions

Powered by Pull Assistant. Last update 9904aff ... 9904aff. Read the comment docs.

Copy link
Contributor

@mgxd mgxd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this looks good, the test (niworkflows.reports.tests.test_core.test_generated_reportlets) needs to be altered as well, specifically the line:

expected_reportlets_num = len(report.layout.get(extension="svg"))

@codecov
Copy link

codecov bot commented Aug 12, 2020

Codecov Report

Merging #548 into master will increase coverage by 5.39%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #548      +/-   ##
==========================================
+ Coverage   59.62%   65.01%   +5.39%     
==========================================
  Files          44       44              
  Lines        5395     5506     +111     
  Branches      789      830      +41     
==========================================
+ Hits         3217     3580     +363     
+ Misses       2026     1766     -260     
- Partials      152      160       +8     
Flag Coverage Δ
#documentation 33.64% <ø> (ø)
#masks ?
#reportlettests 100.00% <ø> (?)
#travis 59.85% <ø> (+4.09%) ⬆️
#unittests 48.29% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
niworkflows/viz/utils.py 80.05% <0.00%> (+0.23%) ⬆️
niworkflows/interfaces/mni.py 42.52% <0.00%> (+0.24%) ⬆️
niworkflows/utils/bids.py 75.00% <0.00%> (+1.73%) ⬆️
niworkflows/interfaces/surf.py 47.46% <0.00%> (+3.26%) ⬆️
niworkflows/interfaces/utils.py 42.75% <0.00%> (+3.56%) ⬆️
niworkflows/interfaces/fixes.py 52.94% <0.00%> (+5.88%) ⬆️
niworkflows/reports/core.py 75.60% <0.00%> (+6.70%) ⬆️
niworkflows/interfaces/ants.py 65.62% <0.00%> (+7.81%) ⬆️
niworkflows/interfaces/cifti.py 53.84% <0.00%> (+10.89%) ⬆️
niworkflows/interfaces/freesurfer.py 49.41% <0.00%> (+11.37%) ⬆️
... and 6 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5767039...9904aff. Read the comment docs.

@mgxd mgxd merged commit 5153624 into nipreps:master Aug 12, 2020
oesteban added a commit that referenced this pull request Aug 12, 2020
oesteban added a commit that referenced this pull request Aug 12, 2020
Also, pin templateflow < 0.6.3 for the same reason.
See #548 for the fix in the development branch.
@effigies effigies deleted the fix/extension branch August 13, 2020 20:20
oesteban added a commit that referenced this pull request Aug 14, 2020
First release in the 1.3.x series. This release includes enhancements and bug-fixes
towards the release of the first LTS version of fMRIPrep.
PyBIDS has been revised to use more recent versions, a series of ANTs' interfaces
have been deemed ready to upstream into Nipype, and several improvements regarding
multi-echo EPI are included.
With thanks to Basile Pinsard for contributions.

  * FIX: Patch ``ApplyTransforms`` spec to permit identity in a chain (#554)
  * FIX: Add dots to extensions in PyBIDS' config file (#548)
  * FIX: Segmentation plots aligned with cardinal axes (#544)
  * FIX: Skip T1w file existence check if ``anat_derivatives`` are provided (#545)
  * FIX: Avoid diverting CIFTI dtype from original BOLD (#532)
  * ENH: Add ``smooth`` input to ``RegridToZooms`` (#549)
  * ENH: Enable ``DerivativesDataSink`` to take multiple source files to derive entities (#547)
  * ENH: Allow ``bold_reference_wf`` to accept multiple EPIs/SBRefs (#408)
  * ENH: Numerical stability of EPI brain-masks using probabilistic prior (#485)
  * ENH: Add a pure-Python interface to resample to specific resolutions (#511)
  * MAINT: Finalize upstreaming of ANTs' interfaces to Nipype (#550)
  * MAINT: Update to Python +3.6 (#541)
HippocampusGirl added a commit to HippocampusGirl/niworkflows that referenced this pull request Sep 29, 2020
1.3.0rc3

First release in the 1.3.x series. This release includes enhancements and bug-fixes
towards the release of the first LTS version of fMRIPrep.
PyBIDS has been revised to use more recent versions, a series of ANTs' interfaces
have been deemed ready to upstream into Nipype, and several improvements regarding
multi-echo EPI are included.
With thanks to Basile Pinsard for contributions.

* FIX: Patch ``ApplyTransforms`` spec to permit identity in a chain (nipreps#554)
* FIX: Add dots to extensions in PyBIDS' config file (nipreps#548)
* FIX: Segmentation plots aligned with cardinal axes (nipreps#544)
* FIX: Skip T1w file existence check if ``anat_derivatives`` are provided (nipreps#545)
* FIX: Avoid diverting CIFTI dtype from original BOLD (nipreps#532)
* ENH: Add ``smooth`` input to ``RegridToZooms`` (nipreps#549)
* ENH: Enable ``DerivativesDataSink`` to take multiple source files to derive entities (nipreps#547)
* ENH: Allow ``bold_reference_wf`` to accept multiple EPIs/SBRefs (nipreps#408)
* ENH: Numerical stability of EPI brain-masks using probabilistic prior (nipreps#485)
* ENH: Add a pure-Python interface to resample to specific resolutions (nipreps#511)
* MAINT: Finalize upstreaming of ANTs' interfaces to Nipype (nipreps#550)
* MAINT: Update to Python +3.6 (nipreps#541)
HippocampusGirl added a commit to HippocampusGirl/niworkflows that referenced this pull request Sep 29, 2020
1.3.0

First release in the 1.3.x series.
This release includes enhancements and bug-fixes towards the release of the first
LTS (*long-term support*) version of *fMRIPrep*.
*PyBIDS* has been revised to use more recent versions, a series of ANTs' interfaces
have been deemed ready to upstream into *Nipype*, and several improvements regarding
multi-echo EPI are included.
With thanks to Basile Pinsard for contributions.

* FIX: Patch ``ApplyTransforms`` spec to permit identity in a chain (nipreps#554)
* FIX: Add dots to extensions in PyBIDS' config file (nipreps#548)
* FIX: Segmentation plots aligned with cardinal axes (nipreps#544)
* FIX: Skip T1w file existence check if ``anat_derivatives`` are provided (nipreps#545)
* FIX: Avoid diverting CIFTI dtype from original BOLD (nipreps#532)
* ENH: Add ``smooth`` input to ``RegridToZooms`` (nipreps#549)
* ENH: Enable ``DerivativesDataSink`` to take multiple source files to derive entities (nipreps#547)
* ENH: Allow ``bold_reference_wf`` to accept multiple EPIs/SBRefs (nipreps#408)
* ENH: Numerical stability of EPI brain-masks using probabilistic prior (nipreps#485)
* ENH: Add a pure-Python interface to resample to specific resolutions (nipreps#511)
* MAINT: Upstream all bug-fixes in the 1.2.9 release
* MAINT: Finalize upstreaming of ANTs' interfaces to Nipype (nipreps#550)
* MAINT: Update to Python +3.6 (nipreps#541)
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