dipy icon indicating copy to clipboard operation
dipy copied to clipboard

NF: Applying Decorators in Module (Align)

Open deka27 opened this issue 1 year ago • 3 comments

This PR is to apply keyword decorator to functions in align module.

This PR is dependent on the main PR, which should be merged first.

Main PR:

  • #3239

deka27 avatar Jun 13 '24 21:06 deka27

Hello @deka27, Thank you for updating !

Cheers ! There are no PEP8 issues in this Pull Request. :beers:

Comment last updated at 2024-07-06 05:06:42 UTC

pep8speaks avatar Jun 13 '24 21:06 pep8speaks

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 83.72%. Comparing base (d3e37d1) to head (161a2e9). Report is 91 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3249      +/-   ##
==========================================
+ Coverage   83.66%   83.72%   +0.06%     
==========================================
  Files         152      152              
  Lines       21436    21524      +88     
  Branches     3472     3472              
==========================================
+ Hits        17934    18022      +88     
  Misses       2637     2637              
  Partials      865      865              
Files Coverage Δ
dipy/align/_public.py 84.69% <100.00%> (+0.82%) :arrow_up:
dipy/align/cpd.py 56.52% <100.00%> (+1.39%) :arrow_up:
dipy/align/imaffine.py 96.00% <100.00%> (+0.10%) :arrow_up:
dipy/align/imwarp.py 95.71% <100.00%> (+0.13%) :arrow_up:
dipy/align/metrics.py 96.18% <100.00%> (+0.10%) :arrow_up:
dipy/align/reslice.py 93.02% <100.00%> (+0.34%) :arrow_up:
dipy/align/scalespace.py 92.80% <100.00%> (+0.15%) :arrow_up:
dipy/align/streamlinear.py 87.77% <100.00%> (+0.33%) :arrow_up:
dipy/align/streamwarp.py 78.78% <100.00%> (+0.66%) :arrow_up:

... and 6 files with indirect coverage changes

codecov[bot] avatar Jun 17 '24 13:06 codecov[bot]

Can you rebase and fix the format since #3239 is merged ? Thanks!

skoudoro avatar Jul 01 '24 15:07 skoudoro