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

ENH: MSMT CSD module unique b-val tolerance parameter improvements #2926

Merged
merged 2 commits into from
Oct 5, 2023

Conversation

jhlegarreta
Copy link
Contributor

@jhlegarreta jhlegarreta commented Oct 5, 2023

  • DOC: Add tol parameter missing docstring to MSMT CSD module methods
  • ENH: Allow unique b-val tolerance parameter to MultiShellDeconvModel

dipy/reconst/mcsd.py Outdated Show resolved Hide resolved
@codecov
Copy link

codecov bot commented Oct 5, 2023

Codecov Report

Merging #2926 (432b380) into master (a46138c) will increase coverage by 0.28%.
Report is 121 commits behind head on master.
The diff coverage is 100.00%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2926      +/-   ##
==========================================
+ Coverage   81.41%   81.69%   +0.28%     
==========================================
  Files         145      146       +1     
  Lines       20147    20347     +200     
  Branches     3217     3226       +9     
==========================================
+ Hits        16402    16623     +221     
+ Misses       2929     2905      -24     
- Partials      816      819       +3     
Files Coverage Δ
dipy/reconst/mcsd.py 86.92% <100.00%> (ø)

... and 47 files with indirect coverage changes

Add `tol` parameter missing docstring to MSMT CSD module methods and
and functions.
Allow the unique b-val tolerance be a parameter to the
`MultiShellDeconvModel` class initialization method.
@jhlegarreta jhlegarreta force-pushed the AddMSMTAutoResponseParamDocstring branch from 54a3f53 to 432b380 Compare October 5, 2023 11:56
@jhlegarreta jhlegarreta changed the title DOC: Add MSMT auto response method tol parameter missing docstring ENH: MSMT CSD module unique b-val tolerance parameter improvements Oct 5, 2023
Copy link
Member

@skoudoro skoudoro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, merging

@skoudoro skoudoro merged commit ea7994f into dipy:master Oct 5, 2023
27 checks passed
@skoudoro
Copy link
Member

skoudoro commented Oct 5, 2023

Thanks @jhlegarreta

@jhlegarreta jhlegarreta deleted the AddMSMTAutoResponseParamDocstring branch October 5, 2023 14:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants