ledatelescope / bifrost

A stream processing framework for high-throughput applications.
BSD 3-Clause "New" or "Revised" License
66 stars 29 forks source link

Fixed buffer resizing in `bifrost.sigproc2` #221

Closed jaycedowell closed 10 months ago

jaycedowell commented 1 year ago

This PR changes how bifrost.sigproc2s internal buffer is resized. Rather than doing an in-place resize (which could cause knock-on problems for other blocks accessing that data) the buffer is re-allocated when a resize is needed. This should be the last task for #214.

codecov[bot] commented 1 year ago

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.63% :tada:

Comparison is base (5a93e5d) 67.56% compared to head (74e620f) 68.19%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #221 +/- ## ========================================== + Coverage 67.56% 68.19% +0.63% ========================================== Files 65 65 Lines 5515 5515 ========================================== + Hits 3726 3761 +35 + Misses 1789 1754 -35 ``` | [Files Changed](https://app.codecov.io/gh/ledatelescope/bifrost/pull/221?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=LEDA+Collaboration) | Coverage Δ | | |---|---|---| | [python/bifrost/sigproc2.py](https://app.codecov.io/gh/ledatelescope/bifrost/pull/221?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=LEDA+Collaboration#diff-cHl0aG9uL2JpZnJvc3Qvc2lncHJvYzIucHk=) | `54.67% <100.00%> (+7.94%)` | :arrow_up: | ... and [4 files with indirect coverage changes](https://app.codecov.io/gh/ledatelescope/bifrost/pull/221/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=LEDA+Collaboration)

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