search

bbfrederick / rapidtide

rapidtide - a suite of programs for doing time lag correlation analysis on fMRI data
Apache License 2.0
75 stars 14 forks source link

Remove unused variables and imports in rapidtide workflow #167

Closed tsalo closed 2 months ago

tsalo commented 2 months ago

Sorry for the unsolicited PR, but it's not a major change. I just removed any variables and imports that my linter flagged as unused. It has a small effect on the memory usage (I get slightly further along in the workflow in my fMRIPost-Rapidtide Docker runs).

codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 75.00000% with 2 lines in your changes missing coverage. Please review.

Project coverage is 35.91%. Comparing base (abe00eb) to head (e5243f1). Report is 3 commits behind head on main.

Files with missing lines Patch % Lines
rapidtide/workflows/rapidtide.py 75.00% 1 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #167 +/- ## ========================================== - Coverage 35.96% 35.91% -0.05% ========================================== Files 129 129 Lines 22518 22501 -17 Branches 3671 3672 +1 ========================================== - Hits 8098 8082 -16 + Misses 13613 13611 -2 - Partials 807 808 +1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.