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

First crack at new shared memory routines #179

Closed bbfrederick closed 3 weeks ago

bbfrederick commented 3 weeks ago

Closes # .

Changes proposed in this pull request: This updates the multiprocessing shared memory routines to use SharedMemory() rather than RawArray(). This is the newer interface, and is supposed to be safer (and I'm hoping will be deallocated properly on linux hosts).

-

codecov[bot] commented 3 weeks ago

Codecov Report

Attention: Patch coverage is 86.27451% with 14 lines in your changes missing coverage. Please review.

Project coverage is 34.57%. Comparing base (b4ec721) to head (119135e). Report is 5 commits behind head on main.

Files with missing lines Patch % Lines
rapidtide/workflows/retroglm.py 0.00% 8 Missing :warning:
rapidtide/workflows/retrolagtcs.py 0.00% 3 Missing :warning:
rapidtide/workflows/rapidtide.py 94.44% 2 Missing :warning:
rapidtide/util.py 93.75% 0 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #179 +/- ## ======================================= Coverage 34.56% 34.57% ======================================= Files 135 136 +1 Lines 24349 24352 +3 Branches 3506 3504 -2 ======================================= + Hits 8417 8419 +2 Misses 15136 15136 - Partials 796 797 +1 ```

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