Skip to content

[FIX] Lock travis pybids 0.6.5 #2720

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 2 commits into from
Oct 9, 2018
Merged

Conversation

adelavega
Copy link
Contributor

This is just a patch to attempt to pin pybids to 0.6.5, until we updates the interfaces that depend on pybids.

@@ -87,8 +87,7 @@ before_install:

- travis_retry pip install -r requirements.txt
- travis_retry pip install grabbit==0.1.2
- travis_retry git clone https://github.com/INCF/pybids.git ${HOME}/pybids &&
pip install -e ${HOME}/pybids
- travis_retry git clone -b 0.6.5 https://github.com/INCF/pybids.git ${HOME}/pybids && pip install -e ${HOME}/pybids
Copy link
Member

Choose a reason for hiding this comment

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

Shorter:

travis_retry pip install -e git+https://github.com/INCF/pybids.git@0.6.5#egg=pybids

@oesteban
Copy link
Contributor

oesteban commented Oct 9, 2018

Shouldn't we also pin the dependency here?

@codecov-io
Copy link

Codecov Report

Merging #2720 into master will decrease coverage by 3.34%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2720      +/-   ##
==========================================
- Coverage   67.61%   64.26%   -3.35%     
==========================================
  Files         340      338       -2     
  Lines       43153    43102      -51     
  Branches     5351     5348       -3     
==========================================
- Hits        29176    27701    -1475     
- Misses      13275    14329    +1054     
- Partials      702     1072     +370
Flag Coverage Δ
#smoketests ?
#unittests 64.26% <ø> (-0.84%) ⬇️
Impacted Files Coverage Δ
nipype/interfaces/nilearn.py 40% <0%> (-56.67%) ⬇️
nipype/utils/spm_docs.py 25.92% <0%> (-44.45%) ⬇️
nipype/interfaces/freesurfer/base.py 49.59% <0%> (-30.9%) ⬇️
nipype/utils/logger.py 59.7% <0%> (-29.86%) ⬇️
nipype/algorithms/rapidart.py 35.39% <0%> (-29.21%) ⬇️
nipype/interfaces/spm/base.py 58.41% <0%> (-29.05%) ⬇️
nipype/utils/provenance.py 55.73% <0%> (-28.99%) ⬇️
nipype/interfaces/fsl/model.py 55.26% <0%> (-25.35%) ⬇️
nipype/interfaces/dipy/setup.py 0% <0%> (-25%) ⬇️
nipype/testing/fixtures.py 77.33% <0%> (-21.34%) ⬇️
... and 43 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 15a8d6f...d7e664f. Read the comment docs.

@adelavega adelavega changed the title Lock travis pybids 0.6.5 [FIX] Lock travis pybids 0.6.5 Oct 9, 2018
@effigies
Copy link
Member

effigies commented Oct 9, 2018

Merging despite all the scary red x's.

cc: @miykael.

@effigies effigies merged commit 88de9f8 into nipy:master Oct 9, 2018
@adelavega adelavega deleted the pybids_travis branch October 9, 2018 19:14
@effigies effigies added this to the 1.1.4 milestone Oct 11, 2018
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.

4 participants