Conversation

yibeichan

out_shell_pe_translation_parameters in fsl.epi.eddy appeared twice, removed the second one to avoid redundancy

@codecov

Codecov Report

coverage has no change and project coverage change: -0.01⚠️

Comparison is base (450be34) 63.34% compared to head (fc8cae7) 63.34%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3568      +/-   ##
==========================================
- Coverage   63.34%   63.34%   -0.01%     
==========================================
  Files         309      309              
  Lines       40904    40903       -1     
  Branches     5381     5381              
==========================================
- Hits        25911    25910       -1     
  Misses      13985    13985              
  Partials     1008     1008              
Impacted FilesCoverage Δ
nipype/interfaces/fsl/epi.py61.88% <ø> (-0.08%)⬇️

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@effigies

Surprised this isn't something flake8 catches.

@effigieseffigies changed the title remove redundant out_shell_pe_translation_parameters STY: Remove duplicated trait in EddyOutputSpec May 10, 2023
@effigieseffigies merged commit 56c8817 into nipy:master May 10, 2023
@yibeichanyibeichan deleted the yibeichan--1 branch May 10, 2023 15:04
@effigieseffigies mentioned this pull request Mar 20, 2024
6 tasks
@effigieseffigies mentioned this pull request Oct 31, 2024
6 tasks
Sign up for free to join this conversation on . Already have an account? Sign in to comment
None yet
None yet

Successfully merging this pull request may close these issues.