Skip to content

Spectral processor#116

Open
kunitoki wants to merge 3 commits into
mainfrom
dev/spectral_processor
Open

Spectral processor#116
kunitoki wants to merge 3 commits into
mainfrom
dev/spectral_processor

Conversation

@kunitoki

@kunitoki kunitoki commented Jun 13, 2026

Copy link
Copy Markdown
Owner

This pull request introduces a new "Spectral Processor" type of audio processor (separated from the Audio Processor but unified at the AudioProcessorBase level) and refactors the audio processor interface to use a unified AudioSpec and SpectralSpec struct for initialization together with AudioProcessContext and SpectralProcessContext, improving code consistency and extensibility via a new templated DomainProcessor<DomainProcessContext, DomainSpecification>. The changes include the implementation and registration of a new audio graph node for processing a passthrough signal using the SpectralBridge class that bridges audio to spectral to audio (which will be the base for future spectral nodes) .

Implements the RFC #101

@codecov

codecov Bot commented Jun 13, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 83.59375% with 84 lines in your changes missing coverage. Please review.
✅ Project coverage is 78.60%. Comparing base (7cfe114) to head (5c364a3).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
...p_audio_processors/spectral/yup_SpectralBridge.cpp 84.44% 35 Missing ⚠️
...udio_processors/spectral/yup_SpectralProcessor.cpp 10.34% 26 Missing ⚠️
...o_processors/processors/yup_AudioProcessorBase.cpp 87.50% 10 Missing ⚠️
..._audio_processors/processors/yup_DomainProcessor.h 56.25% 7 Missing ⚠️
...dio_processors/processors/yup_AudioProcessorBase.h 92.68% 3 Missing ⚠️
...yup_audio_basics/buffers/yup_AudioSpectralBuffer.h 97.77% 2 Missing ⚠️
...yup_audio_processors/spectral/yup_SpectralBridge.h 83.33% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #116      +/-   ##
==========================================
+ Coverage   78.57%   78.60%   +0.03%     
==========================================
  Files         605      614       +9     
  Lines       60567    60956     +389     
==========================================
+ Hits        47592    47916     +324     
- Misses      12975    13040      +65     
Files with missing lines Coverage Δ
.../yup_audio_graph/graph/yup_AudioGraphProcessor.cpp 94.78% <100.00%> (ø)
...es/yup_audio_graph/graph/yup_AudioGraphProcessor.h 0.00% <ø> (ø)
...p_audio_processors/processors/yup_AudioBusLayout.h 100.00% <100.00%> (ø)
...audio_processors/processors/yup_AudioProcessor.cpp 59.37% <100.00%> (-15.19%) ⬇️
...p_audio_processors/processors/yup_AudioProcessor.h 50.00% <ø> (-33.79%) ⬇️
...es/yup_audio_processors/processors/yup_AudioSpec.h 100.00% <100.00%> (ø)
...s/yup_audio_processors/spectral/yup_SpectralSpec.h 100.00% <100.00%> (ø)
...yup_audio_processors/spectral/yup_SpectralBridge.h 83.33% <83.33%> (ø)
...yup_audio_basics/buffers/yup_AudioSpectralBuffer.h 97.77% <97.77%> (ø)
...dio_processors/processors/yup_AudioProcessorBase.h 92.68% <92.68%> (ø)
... and 4 more

... and 6 files with indirect coverage changes


Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 12b7962...5c364a3. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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