Skip to content

Commit f4f791a

Browse files
committed
Fix parameter evaluation
1 parent 9b4f436 commit f4f791a

3 files changed

Lines changed: 1 addition & 24 deletions

File tree

src/plugins/synth/internal/scheduler/SynthesisDocumentWriter.cpp

Lines changed: 1 addition & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -17,10 +17,8 @@
1717
#include <dspxmodelORM/Phoneme.h>
1818
#include <dspxmodelORM/PhonemeSequence.h>
1919
#include <dspxmodelORM/SingingClip.h>
20-
#include <dspxmodelORM/Sources.h>
2120
#include <synth/SynthesisPiece.h>
2221
#include <synth/internal/SynthesisProjectInput.h>
23-
#include <synth/internal/private/ParameterExpressionUtils_p.h>
2422

2523
namespace Synth::Internal::DocumentWriter {
2624

@@ -54,20 +52,12 @@ namespace Synth::Internal::DocumentWriter {
5452
if (!modelParameter || it->values.isEmpty() || it->sampleRate <= 0.0) {
5553
continue;
5654
}
57-
const auto configuration = ProjectInput::parameterConfiguration(clip->sources()->category(), it.key());
5855
QList<QVariant> values;
5956
for (int index = firstIndex; index < lastIndex; ++index) {
6057
const int tick = index * dspx::FreeValueDataArray::step();
6158
const double offset = ProjectInput::tickSeconds(timeline, clip->start() + tick) - pieceStartSeconds;
6259
const int sample = std::clamp(static_cast<int>(std::round(offset * it->sampleRate)), 0, static_cast<int>(it->values.size()) - 1);
63-
double value = it->values.at(sample);
64-
if (!configuration.id().isEmpty()) {
65-
double denormalized{};
66-
if (ParameterExpressionUtils::evaluate(configuration.denormalizationExpression(), value, &denormalized)) {
67-
value = denormalized;
68-
}
69-
}
70-
values.append(static_cast<int>(std::round(value)));
60+
values.append(static_cast<int>(std::round(it->values.at(sample))));
7161
}
7262
auto original = modelParameter->original();
7363
if (original->size() < firstIndex) {

src/plugins/synth/internal/scheduler/SynthesisParameterEvaluator.cpp

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,6 @@
1515
#include <dspxmodelORM/FreeValueDataArray.h>
1616
#include <dspxmodelORM/Parameter.h>
1717
#include <opendspxinterpolator/parameterinterpolator.h>
18-
#include <synth/internal/private/ParameterExpressionUtils_p.h>
1918

2019
namespace Synth::Internal {
2120

@@ -178,13 +177,4 @@ namespace Synth::Internal {
178177
return *base * (transform ? *transform / 1000.0 : 1.0);
179178
}
180179

181-
double SynthesisParameterEvaluator::normalize(const ParameterConfiguration &configuration, double value) {
182-
double normalized = value;
183-
if (!configuration.id().isEmpty() &&
184-
!ParameterExpressionUtils::evaluate(configuration.normalizationExpression(), value, &normalized)) {
185-
normalized = value;
186-
}
187-
return normalized;
188-
}
189-
190180
}

src/plugins/synth/internal/scheduler/SynthesisParameterEvaluator.h

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,6 @@
33

44
#include <memory>
55

6-
#include <synth/ParameterConfiguration.h>
7-
86
namespace dspx {
97
class Parameter;
108
}
@@ -17,7 +15,6 @@ namespace Synth::Internal {
1715
~SynthesisParameterEvaluator();
1816

1917
double evaluate(double tick, double fallback) const;
20-
static double normalize(const ParameterConfiguration &configuration, double value);
2118

2219
private:
2320
struct Data;

0 commit comments

Comments
 (0)