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

Use integer division to avoid errors in indexing #1216

Merged
merged 4 commits into from Apr 6, 2017

Conversation

Projects
None yet
4 participants
@ghoshbishakh
Member

ghoshbishakh commented Apr 6, 2017

Fixes #1215
Fix PEP8 errors

@ghoshbishakh ghoshbishakh force-pushed the ghoshbishakh:fix1215 branch from 9f0eb5f to 4a85a1f Apr 6, 2017

@coveralls

This comment has been minimized.

coveralls commented Apr 6, 2017

Coverage Status

Coverage remained the same at 88.383% when pulling 4a85a1f on ghoshbishakh:fix1215 into 9b344dc on nipy:master.

@coveralls

This comment has been minimized.

coveralls commented Apr 6, 2017

Coverage Status

Coverage remained the same at 88.383% when pulling 4a85a1f on ghoshbishakh:fix1215 into 9b344dc on nipy:master.

@codecov-io

This comment has been minimized.

codecov-io commented Apr 6, 2017

Codecov Report

Merging #1216 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1216   +/-   ##
=======================================
  Coverage   85.87%   85.87%           
=======================================
  Files         221      221           
  Lines       27121    27121           
  Branches     2776     2776           
=======================================
  Hits        23290    23290           
  Misses       3148     3148           
  Partials      683      683

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 9b344dc...33bb686. Read the comment docs.

@coveralls

This comment has been minimized.

coveralls commented Apr 6, 2017

Coverage Status

Coverage remained the same at 88.383% when pulling d30a0e8 on ghoshbishakh:fix1215 into 9b344dc on nipy:master.

@coveralls

This comment has been minimized.

coveralls commented Apr 6, 2017

Coverage Status

Coverage remained the same at 88.383% when pulling 33bb686 on ghoshbishakh:fix1215 into 9b344dc on nipy:master.

@arokem

This comment has been minimized.

Member

arokem commented Apr 6, 2017

Looks good. Merging.

@arokem arokem merged commit 857ca39 into nipy:master Apr 6, 2017

4 checks passed

codecov/patch Coverage not affected when comparing 9b344dc...33bb686
Details
codecov/project 85.87% remains the same compared to 9b344dc
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 88.383%
Details

ShreyasFadnavis pushed a commit to ShreyasFadnavis/dipy that referenced this pull request Sep 20, 2018

Merge pull request nipy#1216 from ghoshbishakh/fix1215
Use integer division to avoid errors in indexing
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment