Skip to content

MRG: test against pre-release numpy / scipy #427

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 5 commits into from
Mar 10, 2017
Merged

MRG: test against pre-release numpy / scipy #427

merged 5 commits into from
Mar 10, 2017

Conversation

matthew-brett
Copy link
Member

Use pre-release numpy / scipy wheels for testing.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 86.683% when pulling 45f226b on matthew-brett:test-pre-wheels into dd147a1 on nipy:master.

@codecov-io
Copy link

codecov-io commented Mar 5, 2017 •
edited
Loading

Codecov Report

Merging #427 into master will decrease coverage by -0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #427      +/-   ##
==========================================
- Coverage   83.27%   83.26%   -0.01%     
==========================================
  Files         301      301              
  Lines       28184    28182       -2     
  Branches     3279     3279              
==========================================
- Hits        23470    23467       -3     
- Misses       3711     3712       +1     
  Partials     1003     1003
Impacted Files Coverage Δ
nipy/algorithms/diagnostics/tsdiffplot.py 78.43% <ø> (-0.82%) ❌
nipy/core/image/image.py 58.73% <ø> (ø) ✅
nipy/algorithms/tests/test_kernel_smooth.py 100% <100%> (ø) ✅
...py/labs/datasets/volumes/tests/test_volume_grid.py 100% <100%> (ø) ✅
...orithms/statistics/tests/test_intrinsic_volumes.py 100% <100%> (ø) ✅
nipy/modalities/fmri/hemodynamic_models.py 96.87% <100%> (ø) ✅
nipy/algorithms/group/parcel_analysis.py 98.02% <100%> (ø) ✅
...algorithms/segmentation/tests/test_segmentation.py 94.52% <100%> (ø) ✅
nipy/algorithms/statistics/formula/formulae.py 89.82% <100%> (ø) ✅
...lgorithms/statistics/formula/tests/test_formula.py 99.32% <0%> (-0.34%) ❌

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 dd147a1...0679948. Read the comment docs.

@matthew-brett
MAINT: test against pre-release numpy / scipy
9b46ba6 
Use pre-release numpy / scipy wheels for testing.
@coveralls
Copy link

Coverage Status

Coverage remained the same at 86.683% when pulling 0d069c7 on matthew-brett:test-pre-wheels into dd147a1 on nipy:master.

@matthew-brett
BF: rewrite boolean subtract as mask inversion
d56e1fb 
Some instances of ``True - msk`` for inverting the mask were causing
errors for numpy pre-release tests.  Use ``~msk`` instead.
@matthew-brett
BF: remove redundant hold calls in tsdiffplot
51745fd 
Axis hold calls were expressing the default already - hold(True).  Hold
calls now deprecated in Matplotlib.
@coveralls
Copy link

Coverage Status

Coverage remained the same at 86.683% when pulling d56e1fb on matthew-brett:test-pre-wheels into dd147a1 on nipy:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.0009%) to 86.682% when pulling 51745fd on matthew-brett:test-pre-wheels into dd147a1 on nipy:master.

@matthew-brett
RF: replace random_integers with randint
f23ae98 
random_integers now deprecated.
@matthew-brett
RF: remove some numpy deprecation warnings
0679948 
Non integer in linspace; error for comparision of arrays with different
shapes.
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.004%) to 86.678% when pulling 0679948 on matthew-brett:test-pre-wheels into dd147a1 on nipy:master.

@matthew-brett matthew-brett changed the title WIP: test against pre-release numpy / scipy MRG: test against pre-release numpy / scipy Mar 5, 2017
@matthew-brett
Copy link
Member Author

Any comments here?

@matthew-brett matthew-brett merged commit 281276b into nipy:master Mar 10, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@matthew-brett @coveralls @codecov-io