Skip to content
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

initial spectrum analyser implementation #29

Merged
merged 34 commits into from
Nov 29, 2023
Merged

Conversation

RachelMaryamLocke
Copy link
Contributor

@RachelMaryamLocke RachelMaryamLocke commented Oct 10, 2023

currently added to 8-band EQ only

Screenshot 2023-10-10 at 23 34 20

after most recent changes:

Screenshot 2023-10-17 at 22 43 45 Screenshot 2023-10-17 at 22 44 16

@RachelMaryamLocke RachelMaryamLocke changed the title initial spectrum analyser commit initial spectrum analyser implementation Oct 10, 2023
src/dsp/EQ/EQProcessor.cpp Outdated Show resolved Hide resolved
src/dsp/Shared/EQHelpers.cpp Outdated Show resolved Hide resolved
Copy link
Contributor

@jatinchowdhury18 jatinchowdhury18 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good on my end, just a few little notes.

src/CMakeLists.txt Outdated Show resolved Hide resolved
src/gui/EQ/EQEditor.cpp Outdated Show resolved Hide resolved
src/gui/PluginEditor.h Outdated Show resolved Hide resolved
src/gui/SVF/SVFPlot.cpp Outdated Show resolved Hide resolved
src/gui/Shared/SpectrumAnalyser.h Outdated Show resolved Hide resolved
src/gui/Shared/SpectrumAnalyserTask.cpp Outdated Show resolved Hide resolved
src/gui/Shared/SpectrumAnalyserTask.h Show resolved Hide resolved
src/gui/Shared/SpectrumAnalyserTask.h Outdated Show resolved Hide resolved
@RachelMaryamLocke RachelMaryamLocke merged commit 168b876 into main Nov 29, 2023
@RachelMaryamLocke RachelMaryamLocke deleted the spectrum_analyser branch November 29, 2023 12:29
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.

2 participants