FIX: Set networkx max version until 2.0 fixes are finalized

[effigies]

Fixes #2194.

Changes proposed in this pull request

• Set networkx max version of 1.11

[effigies]

Looks like Travis just really hates afni.Hist all the sudden. So much for this working first.

[satra]

i can't see anything in the code that would be weird. for the time being could you temporarily remove that particular test just to check travis?

also since circle runs on py36 with dependencies, it's weird that the error condition on travis is on the closest cell that circle runs on.

[effigies]

@satra Note that all INSTALL_DEB_DEPENDENCIES=true lines are failing. Wonder if it's a neurodebian problem.

https://travis-ci.org/nipy/nipype/builds/278944845

[effigies]

What test is actually failing? The doctest?

[codecov-io]

Codecov Report

Merging #2195 into master will decrease coverage by 30.12%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           master    #2195       +/-   ##
===========================================
- Coverage   72.28%   42.16%   -30.13%     
===========================================
  Files        1171       68     -1103     
  Lines       58622    14497    -44125     
  Branches     8433        0     -8433     
===========================================
- Hits        42377     6113    -36264     
+ Misses      14887     8384     -6503     
+ Partials     1358        0     -1358

Flag	Coverage Δ
#smoketests	42.16% <100%> (-30.13%)	⬇️
#unittests	?

Impacted Files	Coverage Δ
nipype/info.py	84.61% <100%> (+0.24%)	⬆️
nipype/pipeline/plugins/callback_log.py	12% <0%> (-80%)	⬇️
nipype/interfaces/utility/wrappers.py	20.37% <0%> (-65.75%)	⬇️
nipype/pipeline/plugins/linear.py	22.85% <0%> (-65.72%)	⬇️
nipype/pipeline/plugins/pbsgraph.py	29.41% <0%> (-64.71%)	⬇️
nipype/pipeline/engine/nodes.py	23.3% <0%> (-61.59%)	⬇️
nipype/algorithms/modelgen.py	15.11% <0%> (-58.38%)	⬇️
nipype/interfaces/utility/csv.py	42.85% <0%> (-57.15%)	⬇️
nipype/interfaces/spm/base.py	29.18% <0%> (-55.52%)	⬇️
nipype/interfaces/utility/base.py	43.25% <0%> (-47.76%)	⬇️
... and 1132 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 4a0bd3d...3ee5f24. Read the comment docs.

[satra]

re: doctest - i believe so.

neurodebian has not updated it since 2016, so i don't know what's going on there.

[effigies]

Now it's skullstrip. Which I think are the two interfaces with redirect_x.

[satra]

nothing is running though in those tests, just an initialization. i wonder whether the afni_vcheck command is failing. trying out now in a debian:stretch container.

[effigies]

I think that's it. I can't hit https://afni.nimh.nih.gov/

[satra]

just verified locally that it's afni_vcheck that's failing.

[effigies]

I think adding a <30s timeout would be a good idea. Nipype shouldn't depend on the AFNI web server's availability.

…

On Sep 23, 2017 1:04 PM, "Satrajit Ghosh" ***@***.***> wrote: just verified locally that it's afni_vcheck that's failing. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#2195 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAFF8s5PAL64e2ZoP1BStJoryT0T6OP2ks5slToEgaJpZM4PhjpD> .

[satra]

timeout exists in subprocess for python 3, but python 2 needs some extra work.

[effigies]

Is there another way to get the version? I couldn't find a flag to avoid fetching the url.

…

On Sep 23, 2017 1:22 PM, "Satrajit Ghosh" ***@***.***> wrote: timeout exists in subprocess for python 3, but python 2 needs some extra work. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#2195 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAFF8gpxMmyC2suzDfYVWbentPAuBxReks5slT5vgaJpZM4PhjpD> .

[satra]

i think this should fix it: see: djarecka#3

[effigies]

Looks reasonable, but I think Dorota is afk this weekend. Maybe just create a PR from your branch?

…

On Sep 23, 2017 2:04 PM, "Satrajit Ghosh" ***@***.***> wrote: i think this should fix it: see: djarecka#3 <djarecka#3> — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#2195 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAFF8rjOk0SHjnf7CDUD5KBiDjWUo3rUks5slUgbgaJpZM4PhjpD> .

[effigies]

Closing in favor of #2196.