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

Cleaned PR for Affine Registration Workflow with Supporting Quality Metrics #1604

wants to merge 14 commits into from


Copy link

commented Jul 31, 2018

This work was done as part of the Google Summer of Code (GSOC-2018) DIPY project titled 'DIPY Workflow(s) and Quality Assurance'

This work pertains to developing and testing the Affine registration workflow for overlaying bio-medical images in different modes (center of mass, translation, rigid body and affine registration). The modes of registration can be controlled easily via the command line parameters and the relative impact on the results can be assessed quantitatively via the generated output metrics. More information in the metrics can be seen in the code documentation as well as by trying the '-h' or help of the workflow.

This PR contains the stable and clean code for affine registration workflow. This should be merged in place of PR 1581. PR 1581 must not be merged now.

The original PR 1581 with the conversation history, all the original commits, commits for improvement based on community feedback can be seen below:


This comment has been minimized.

Copy link

commented Jul 31, 2018

Hello @parichit, Thank you for updating !

Line 146:9: E303 too many blank lines (2)

Comment last updated on August 14, 2018 at 06:00 Hours UTC

@skoudoro skoudoro added the gsoc2018 label Aug 1, 2018

@parichit parichit force-pushed the parichit:affine_registration_clean branch from e951e87 to d55dfb2 Aug 7, 2018

Decreased the decimal precision to 1 decimal places (in checking the …
…value of distance metric) to validate the tests by travis.

This comment has been minimized.

Copy link

commented Aug 8, 2018

Codecov Report

Merging #1604 into master will increase coverage by 0.02%.
The diff coverage is 97.5%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1604      +/-   ##
+ Coverage   87.34%   87.37%   +0.02%     
  Files         246      246              
  Lines       31811    31943     +132     
  Branches     3451     3462      +11     
+ Hits        27785    27909     +124     
- Misses       3204     3207       +3     
- Partials      822      827       +5
Impacted Files Coverage Δ
dipy/align/ 91.84% <100%> (+0.04%) ⬆️
dipy/workflows/ 90.12% <100%> (-3.63%) ⬇️
dipy/io/ 100% <100%> (ø) ⬆️
dipy/workflows/tests/ 97.46% <96.87%> (+6.55%) ⬆️
dipy/core/ 73.8% <0%> (-1.2%) ⬇️

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 5a6aa5a...cf40e88. Read the comment docs."Similarity metric:"+str(xopt))"Distance measure:"+str(fopt))

This comment has been minimized.

Copy link

Garyfallidis Aug 8, 2018


Optimal parameters
Similarity metric

parichit added 3 commits Aug 13, 2018
1) Updated the parameter documentation for optimal parameters and sim…
…ilarity metric in the optimize() function of the file.

2) Changed the parameter documentation in the run() method of the alignpy file for improved view on the command line.

This comment has been minimized.

Copy link

commented Feb 7, 2019

Closing in favor of #1735. Thanks @parichit for this work.

@skoudoro skoudoro closed this Feb 7, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
5 participants
You can’t perform that action at this time.