Skip to content
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 in reconst_gqi.py #1217

Merged
merged 6 commits into from Apr 11, 2017
Merged

Conversation

ghoshbishakh
Copy link
Contributor

No description provided.

@ghoshbishakh
Copy link
Contributor Author

lets check all the examples before merging this one

@codecov-io
Copy link

codecov-io commented Apr 7, 2017

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1217   +/-   ##
=======================================
  Coverage   85.91%   85.91%           
=======================================
  Files         221      221           
  Lines       27247    27247           
  Branches     2782     2782           
=======================================
  Hits        23408    23408           
  Misses       3154     3154           
  Partials      685      685

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 f6c8dee...586909a. Read the comment docs.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 88.383% when pulling 15c6f38 on ghoshbishakh:fix1215 into 857ca39 on nipy:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 88.383% when pulling 1112bdc on ghoshbishakh:fix1215 into 857ca39 on nipy:master.

@arokem
Copy link
Contributor

arokem commented Apr 9, 2017

Just checking: have you had a chance to check the other examples? I propose that add the changes to the other examples on this same PR.

@ghoshbishakh
Copy link
Contributor Author

ghoshbishakh commented Apr 11, 2017

@arokem no still a few are left.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 88.415% when pulling f9a0b5d on ghoshbishakh:fix1215 into f6c8dee on nipy:master.

@ghoshbishakh
Copy link
Contributor Author

@arokem All examples are working now! Ready to merge.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 88.415% when pulling 586909a on ghoshbishakh:fix1215 into f6c8dee on nipy:master.

@arokem
Copy link
Contributor

arokem commented Apr 11, 2017

Awesome. Thanks for doing this!

@arokem arokem merged commit bdee770 into dipy:master Apr 11, 2017
ShreyasFadnavis pushed a commit to ShreyasFadnavis/dipy that referenced this pull request Sep 20, 2018
Use integer division in reconst_gqi.py
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.

None yet

4 participants