Skip to content

Span API support #51

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

Open
wants to merge 12 commits into
base: develop
Choose a base branch
from
Open

Span API support #51

wants to merge 12 commits into from

Conversation

vitalsong
Copy link
Owner

  • Remove auto cast arr_real -> arr_cmplx;
  • Add complex(re) function;
  • Add span implementation;
  • Return span from slice function for step=1;
  • Remove base_array reference from slice;
  • Check same array by pointer range;
  • Simplify cast for cmplx_t type;
  • Add span instead base_array for some math functions;
  • Update tests;

@vitalsong vitalsong force-pushed the feature/span-support branch from 762b60b to 907a53f Compare April 29, 2024 16:50
@vitalsong vitalsong changed the base branch from master to develop April 29, 2024 16:50
@vitalsong vitalsong force-pushed the develop branch 2 times, most recently from 0076124 to daf6171 Compare May 25, 2024 16:36
@vitalsong vitalsong force-pushed the feature/span-support branch from 907a53f to 79aeeb3 Compare May 26, 2024 09:46
@vitalsong vitalsong changed the base branch from develop to master May 26, 2024 09:46
Copy link

codecov bot commented May 26, 2024

Codecov Report

Attention: Patch coverage is 97.10145% with 10 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
lib/fft/primes-fft.h 73.33% 4 Missing ⚠️
lib/fft.cpp 66.66% 2 Missing ⚠️
lib/fft/fact-fft.h 60.00% 2 Missing ⚠️
include/dsplib/czt.h 0.00% 1 Missing ⚠️
include/dsplib/span.h 98.30% 1 Missing ⚠️
Files with missing lines Coverage Δ
include/dsplib/array.h 99.26% <100.00%> (+0.03%) ⬆️
include/dsplib/fft.h 100.00% <ø> (+75.00%) ⬆️
include/dsplib/ifft.h 100.00% <ø> (ø)
include/dsplib/math.h 86.36% <ø> (ø)
include/dsplib/slice.h 98.85% <100.00%> (+6.25%) ⬆️
include/dsplib/types.h 100.00% <ø> (ø)
include/dsplib/utils.h 78.57% <ø> (-7.85%) ⬇️
lib/czt.cpp 90.24% <100.00%> (+0.77%) ⬆️
lib/fft/cmplx-ifft.h 89.47% <100.00%> (+4.85%) ⬆️
lib/fft/fact-fft.cpp 98.94% <100.00%> (+2.28%) ⬆️
... and 18 more
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@vitalsong
Copy link
Owner Author

TODO: add span iterators

@vitalsong vitalsong changed the base branch from master to develop April 23, 2025 10:34
- Remove auto cast arr_real -> arr_cmplx;
- Add `complex(re)` function;
- Add `span` implementation;
- Return `span` from `slice` function for step=1;
- Remove `base_array` reference from `slice`;
- Check same array by pointer range;
- Simplify cast for `cmplx_t` type;
- Add `span` instead `base_array` for some `math` functions;
- Update tests;
@vitalsong vitalsong force-pushed the feature/span-support branch from b4514df to f646656 Compare April 23, 2025 14:06
@vitalsong vitalsong force-pushed the feature/span-support branch from f646656 to 52fd89b Compare April 23, 2025 14:10
@vitalsong vitalsong force-pushed the feature/span-support branch from 978323b to 66669f2 Compare April 29, 2025 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant