Skip to content

Commit

Permalink
InstrumentSoundShaping refactoring (#7229)
Browse files Browse the repository at this point in the history 
* Accessors for volume, cutoff, resonance

Add private accessors for the volume, cutoff and resonance parameters (envelope and LFO).

This makes the code less technical and more readable as it for example removes lots of static casts. The casts are now done in a special helper method that returns the parameters for a given target.

* Remove EnvelopeAndLfoParameters array

Remove the `EnvelopeAndLfoParameters` array and use explicit instances for volume, cutoff, resonance. This simplifies construction and initialization of the instances.

Besides the array this also removes the `Target` enum and the `NumTargets` value.

To simplify storage and retrieval of the parameters three private methods have been added which provide the node names of the volume, cutoff and resonance parameters.

Adjust `InstrumentSoundShapingView` to the removed `NumTargets` property.

* Use references instead of pointers

Use references to the volume, cutoff and resonance parameters instead of pointers.

* Remove friend relationship

Remove the friend relationship between `InstrumentSoundShaping` and its related view by providing the models via getters.

* Get rid of targetNames

Get rid of `InstrumentSoundShaping::targetNames` by using translations and strings directly. Move the remaining stuff into `InstrumentSoundShapingView` until it is removed there as well.

* Explicit EnvelopeAndLfoViews

Remove the array of EnvelopeAndLfoViews and use dedicated instances instead.

This also enables the final removal of the remaining `targetNames`.

* Move the code of some getters

Move the code of some getters into the header file.

* Several code review changes

Apply some code review proposals.

Co-authored-by: Sotonye Atemie <[email protected]>

---------

Co-authored-by: Sotonye Atemie <[email protected]>
  • Loading branch information
@michaelgregorius @sakertooth
michaelgregorius and sakertooth authored Mar 9, 2025
1 parent f44aa3e commit 8821d88
Show file tree
Hide file tree
Showing 4 changed files with 124 additions and 106 deletions.
40 changes: 23 additions & 17 deletions include/InstrumentSoundShaping.h
View file
Original file line number Diff line number Diff line change
Expand Up @@ -26,13 +26,13 @@
#define LMMS_INSTRUMENT_SOUND_SHAPING_H

#include "ComboBoxModel.h"
#include "EnvelopeAndLfoParameters.h"

namespace lmms
{


class InstrumentTrack;
class EnvelopeAndLfoParameters;
class NotePlayHandle;
class SampleFrame;

Expand All @@ -52,14 +52,19 @@ class InstrumentSoundShaping : public Model, public JournallingObject
void processAudioBuffer( SampleFrame* _ab, const fpp_t _frames,
NotePlayHandle * _n );

enum class Target
{
Volume,
Cut,
Resonance,
Count
} ;
constexpr static auto NumTargets = static_cast<std::size_t>(Target::Count);
const EnvelopeAndLfoParameters& getVolumeParameters() const { return m_volumeParameters; }
EnvelopeAndLfoParameters& getVolumeParameters() { return m_volumeParameters; }

const EnvelopeAndLfoParameters& getCutoffParameters() const { return m_cutoffParameters; }
EnvelopeAndLfoParameters& getCutoffParameters() { return m_cutoffParameters; }

const EnvelopeAndLfoParameters& getResonanceParameters() const { return m_resonanceParameters; }
EnvelopeAndLfoParameters& getResonanceParameters() { return m_resonanceParameters; }

BoolModel& getFilterEnabledModel() { return m_filterEnabledModel; }
ComboBoxModel& getFilterModel() { return m_filterModel; }
FloatModel& getFilterCutModel() { return m_filterCutModel; }
FloatModel& getFilterResModel() { return m_filterResModel; }

f_cnt_t envFrames( const bool _only_vol = false ) const;
f_cnt_t releaseFrames() const;
Expand All @@ -74,22 +79,23 @@ class InstrumentSoundShaping : public Model, public JournallingObject
return "eldata";
}

private:
QString getVolumeNodeName() const;
QString getCutoffNodeName() const;
QString getResonanceNodeName() const;

private:
EnvelopeAndLfoParameters * m_envLfoParameters[NumTargets];
InstrumentTrack * m_instrumentTrack;

EnvelopeAndLfoParameters m_volumeParameters;
EnvelopeAndLfoParameters m_cutoffParameters;
EnvelopeAndLfoParameters m_resonanceParameters;

BoolModel m_filterEnabledModel;
ComboBoxModel m_filterModel;
FloatModel m_filterCutModel;
FloatModel m_filterResModel;

static const char *const targetNames[NumTargets][3];


friend class gui::InstrumentSoundShapingView;

} ;
};


} // namespace lmms
Expand Down
8 changes: 5 additions & 3 deletions include/InstrumentSoundShapingView.h
View file
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,10 @@ class InstrumentSoundShapingView : public QWidget, public ModelView

InstrumentSoundShaping * m_ss = nullptr;
TabWidget * m_targetsTabWidget;
EnvelopeAndLfoView * m_envLfoViews[InstrumentSoundShaping::NumTargets];

EnvelopeAndLfoView* m_volumeView;
EnvelopeAndLfoView* m_cutoffView;
EnvelopeAndLfoView* m_resonanceView;

// filter-stuff
GroupBox * m_filterGroupBox;
Expand All @@ -67,8 +70,7 @@ class InstrumentSoundShapingView : public QWidget, public ModelView
Knob * m_filterResKnob;

QLabel* m_singleStreamInfoLabel;

} ;
};


} // namespace lmms::gui
Expand Down
153 changes: 81 additions & 72 deletions src/core/InstrumentSoundShaping.cpp
View file
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,6 @@
#include "BasicFilters.h"
#include "embed.h"
#include "Engine.h"
#include "EnvelopeAndLfoParameters.h"
#include "Instrument.h"
#include "InstrumentTrack.h"

Expand All @@ -43,42 +42,21 @@ const float RES_MULTIPLIER = 2.0f;
const float RES_PRECISION = 1000.0f;


// names for env- and lfo-targets - first is name being displayed to user
// and second one is used internally, e.g. for saving/restoring settings
const char *const InstrumentSoundShaping::targetNames[InstrumentSoundShaping::NumTargets][3] =
{
{ QT_TRANSLATE_NOOP("InstrumentSoundShaping", "VOLUME"), "vol",
QT_TRANSLATE_NOOP("InstrumentSoundShaping", "Volume") },
{ QT_TRANSLATE_NOOP("InstrumentSoundShaping", "CUTOFF"), "cut",
QT_TRANSLATE_NOOP("InstrumentSoundShaping", "Cutoff frequency") },
{ QT_TRANSLATE_NOOP("InstrumentSoundShaping", "RESO"), "res",
QT_TRANSLATE_NOOP("InstrumentSoundShaping", "Resonance") }
} ;



InstrumentSoundShaping::InstrumentSoundShaping(
InstrumentTrack * _instrument_track ) :
Model( _instrument_track, tr( "Envelopes/LFOs" ) ),
m_instrumentTrack( _instrument_track ),
m_volumeParameters(1., this),
m_cutoffParameters(0., this),
m_resonanceParameters(0., this),
m_filterEnabledModel( false, this ),
m_filterModel( this, tr( "Filter type" ) ),
m_filterCutModel( 14000.0, 1.0, 14000.0, 1.0, this, tr( "Cutoff frequency" ) ),
m_filterResModel(0.5f, BasicFilters<>::minQ(), 10.f, 0.01f, this, tr("Q/Resonance"))
{
for (auto i = std::size_t{0}; i < NumTargets; ++i)
{
float value_for_zero_amount = 0.0;
if( static_cast<Target>(i) == Target::Volume )
{
value_for_zero_amount = 1.0;
}
m_envLfoParameters[i] = new EnvelopeAndLfoParameters(
value_for_zero_amount,
this );
m_envLfoParameters[i]->setDisplayName(
tr( targetNames[i][2] ) );
}
m_volumeParameters.setDisplayName(tr("Volume"));
m_cutoffParameters.setDisplayName(tr("Cutoff frequency"));
m_resonanceParameters.setDisplayName(tr("Resonance"));

m_filterModel.addItem( tr( "Low-pass" ), std::make_unique<PixmapLoader>( "filter_lp" ) );
m_filterModel.addItem( tr( "Hi-pass" ), std::make_unique<PixmapLoader>( "filter_hp" ) );
Expand Down Expand Up @@ -119,7 +97,7 @@ float InstrumentSoundShaping::volumeLevel( NotePlayHandle* n, const f_cnt_t fram
}

float level;
m_envLfoParameters[static_cast<std::size_t>(Target::Volume)]->fillLevel( &level, frame, envReleaseBegin, 1 );
getVolumeParameters().fillLevel(&level, frame, envReleaseBegin, 1);

return level;
}
Expand Down Expand Up @@ -148,6 +126,9 @@ void InstrumentSoundShaping::processAudioBuffer( SampleFrame* buffer,

// only use filter, if it is really needed

auto& cutoffParameters = getCutoffParameters();
auto& resonanceParameters = getResonanceParameters();

if( m_filterEnabledModel.value() )
{
QVarLengthArray<float> cutBuffer(frames);
Expand All @@ -162,20 +143,20 @@ void InstrumentSoundShaping::processAudioBuffer( SampleFrame* buffer,
}
n->m_filter->setFilterType( static_cast<BasicFilters<>::FilterType>(m_filterModel.value()) );

if( m_envLfoParameters[static_cast<std::size_t>(Target::Cut)]->isUsed() )
if (cutoffParameters.isUsed())
{
m_envLfoParameters[static_cast<std::size_t>(Target::Cut)]->fillLevel( cutBuffer.data(), envTotalFrames, envReleaseBegin, frames );
cutoffParameters.fillLevel(cutBuffer.data(), envTotalFrames, envReleaseBegin, frames);
}
if( m_envLfoParameters[static_cast<std::size_t>(Target::Resonance)]->isUsed() )

if (resonanceParameters.isUsed())
{
m_envLfoParameters[static_cast<std::size_t>(Target::Resonance)]->fillLevel( resBuffer.data(), envTotalFrames, envReleaseBegin, frames );
resonanceParameters.fillLevel(resBuffer.data(), envTotalFrames, envReleaseBegin, frames);
}

const float fcv = m_filterCutModel.value();
const float frv = m_filterResModel.value();

if( m_envLfoParameters[static_cast<std::size_t>(Target::Cut)]->isUsed() &&
m_envLfoParameters[static_cast<std::size_t>(Target::Resonance)]->isUsed() )
if (cutoffParameters.isUsed() && resonanceParameters.isUsed())
{
for( fpp_t frame = 0; frame < frames; ++frame )
{
Expand All @@ -196,7 +177,7 @@ void InstrumentSoundShaping::processAudioBuffer( SampleFrame* buffer,
buffer[frame][1] = n->m_filter->update( buffer[frame][1], 1 );
}
}
else if( m_envLfoParameters[static_cast<std::size_t>(Target::Cut)]->isUsed() )
else if (cutoffParameters.isUsed())
{
for( fpp_t frame = 0; frame < frames; ++frame )
{
Expand All @@ -213,7 +194,7 @@ void InstrumentSoundShaping::processAudioBuffer( SampleFrame* buffer,
buffer[frame][1] = n->m_filter->update( buffer[frame][1], 1 );
}
}
else if( m_envLfoParameters[static_cast<std::size_t>(Target::Resonance)]->isUsed() )
else if(resonanceParameters.isUsed() )
{
for( fpp_t frame = 0; frame < frames; ++frame )
{
Expand Down Expand Up @@ -241,10 +222,12 @@ void InstrumentSoundShaping::processAudioBuffer( SampleFrame* buffer,
}
}

if( m_envLfoParameters[static_cast<std::size_t>(Target::Volume)]->isUsed() )
auto& volumeParameters = getVolumeParameters();

if (volumeParameters.isUsed())
{
QVarLengthArray<float> volBuffer(frames);
m_envLfoParameters[static_cast<std::size_t>(Target::Volume)]->fillLevel( volBuffer.data(), envTotalFrames, envReleaseBegin, frames );
volumeParameters.fillLevel(volBuffer.data(), envTotalFrames, envReleaseBegin, frames);

for( fpp_t frame = 0; frame < frames; ++frame )
{
Expand Down Expand Up @@ -275,19 +258,23 @@ void InstrumentSoundShaping::processAudioBuffer( SampleFrame* buffer,

f_cnt_t InstrumentSoundShaping::envFrames( const bool _only_vol ) const
{
f_cnt_t ret_val = m_envLfoParameters[static_cast<std::size_t>(Target::Volume)]->PAHD_Frames();
f_cnt_t ret_val = getVolumeParameters().PAHD_Frames();

if( _only_vol == false )
if (!_only_vol)
{
for (auto i = static_cast<std::size_t>(Target::Volume) + 1; i < NumTargets; ++i)
auto& cutoffParameters = getCutoffParameters();
if (cutoffParameters.isUsed())
{
if( m_envLfoParameters[i]->isUsed() &&
m_envLfoParameters[i]->PAHD_Frames() > ret_val )
{
ret_val = m_envLfoParameters[i]->PAHD_Frames();
}
ret_val = std::max(ret_val, cutoffParameters.PAHD_Frames());
}

auto& resonanceParameters = getResonanceParameters();
if (resonanceParameters.isUsed())
{
ret_val = std::max(ret_val, resonanceParameters.PAHD_Frames());
}
}

return ret_val;
}

Expand All @@ -308,23 +295,33 @@ f_cnt_t InstrumentSoundShaping::releaseFrames() const
return ret_val;
}

if( m_envLfoParameters[static_cast<std::size_t>(Target::Volume)]->isUsed() )
auto& volumeParameters = getVolumeParameters();

if (volumeParameters.isUsed())
{
return m_envLfoParameters[static_cast<std::size_t>(Target::Volume)]->releaseFrames();
return volumeParameters.releaseFrames();
}

for (auto i = static_cast<std::size_t>(Target::Volume) + 1; i < NumTargets; ++i)
auto& cutoffParameters = getCutoffParameters();
if (cutoffParameters.isUsed())
{
if( m_envLfoParameters[i]->isUsed() )
{
ret_val = std::max(ret_val, m_envLfoParameters[i]->releaseFrames());
}
ret_val = std::max(ret_val, cutoffParameters.releaseFrames());
}

auto& resonanceParameters = getResonanceParameters();
if (resonanceParameters.isUsed())
{
ret_val = std::max(ret_val, resonanceParameters.releaseFrames());
}

return ret_val;
}



static void saveEnvelopeAndLFOParameters(EnvelopeAndLfoParameters& p, const QString & tagName, QDomDocument & _doc, QDomElement & _this)
{
p.saveState(_doc, _this).setTagName(tagName);
}

void InstrumentSoundShaping::saveSettings( QDomDocument & _doc, QDomElement & _this )
{
Expand All @@ -333,12 +330,9 @@ void InstrumentSoundShaping::saveSettings( QDomDocument & _doc, QDomElement & _t
m_filterResModel.saveSettings( _doc, _this, "fres" );
m_filterEnabledModel.saveSettings( _doc, _this, "fwet" );

for (auto i = std::size_t{0}; i < NumTargets; ++i)
{
m_envLfoParameters[i]->saveState( _doc, _this ).setTagName(
m_envLfoParameters[i]->nodeName() +
QString( targetNames[i][1] ).toLower() );
}
saveEnvelopeAndLFOParameters(getVolumeParameters(), getVolumeNodeName(), _doc, _this);
saveEnvelopeAndLFOParameters(getCutoffParameters(), getCutoffNodeName(), _doc, _this);
saveEnvelopeAndLFOParameters(getResonanceParameters(), getResonanceNodeName(), _doc, _this);
}


Expand All @@ -352,27 +346,42 @@ void InstrumentSoundShaping::loadSettings( const QDomElement & _this )
m_filterEnabledModel.loadSettings( _this, "fwet" );

QDomNode node = _this.firstChild();
while( !node.isNull() )
while (!node.isNull())
{
if( node.isElement() )
if (node.isElement())
{
for (auto i = std::size_t{0}; i < NumTargets; ++i)
const auto nodeName = node.nodeName();
if (nodeName == getVolumeNodeName())
{
if( node.nodeName() ==
m_envLfoParameters[i]->nodeName() +
QString( targetNames[i][1] ).
toLower() )
{
m_envLfoParameters[i]->restoreState( node.toElement() );
}
getVolumeParameters().restoreState(node.toElement());
}
else if (nodeName == getCutoffNodeName())
{
getCutoffParameters().restoreState(node.toElement());
}
else if (nodeName == getResonanceNodeName())
{
getResonanceParameters().restoreState(node.toElement());
}
}

node = node.nextSibling();
}
}

QString InstrumentSoundShaping::getVolumeNodeName() const
{
return getVolumeParameters().nodeName() + "vol";
}

QString InstrumentSoundShaping::getCutoffNodeName() const
{
return getCutoffParameters().nodeName() + "cut";
}


QString InstrumentSoundShaping::getResonanceNodeName() const
{
return getResonanceParameters().nodeName() + "res";
}

} // namespace lmms
Loading

0 comments on commit 8821d88

Please sign in to comment.