Add possibility to use int variables as parameters

In addition to floats.
This commit is contained in:
Oleg Kalachev
2026-01-18 21:24:44 +03:00
parent 9a9bd07251
commit 6548ae5708

View File

@@ -14,8 +14,13 @@ Preferences storage;
struct Parameter { struct Parameter {
const char *name; // max length is 15 (Preferences key limit) const char *name; // max length is 15 (Preferences key limit)
float *variable; bool integer;
float value; // cache union { float *f; int *i; }; // pointer to variable
float cache; // what's stored in flash
Parameter(const char *name, float *variable) : name(name), integer(false), f(variable) {};
Parameter(const char *name, int *variable) : name(name), integer(true), i(variable) {};
float getValue() const { return integer ? *i : *f; };
void setValue(const float value) { if (integer) *i = value; else *f = value; };
}; };
Parameter parameters[] = { Parameter parameters[] = {
@@ -84,10 +89,10 @@ void setupParameters() {
// Read parameters from storage // Read parameters from storage
for (auto &parameter : parameters) { for (auto &parameter : parameters) {
if (!storage.isKey(parameter.name)) { if (!storage.isKey(parameter.name)) {
storage.putFloat(parameter.name, *parameter.variable); storage.putFloat(parameter.name, parameter.getValue()); // store default value
} }
*parameter.variable = storage.getFloat(parameter.name, *parameter.variable); parameter.setValue(storage.getFloat(parameter.name, 0));
parameter.value = *parameter.variable; parameter.cache = parameter.getValue();
} }
} }
@@ -102,13 +107,13 @@ const char *getParameterName(int index) {
float getParameter(int index) { float getParameter(int index) {
if (index < 0 || index >= parametersCount()) return NAN; if (index < 0 || index >= parametersCount()) return NAN;
return *parameters[index].variable; return parameters[index].getValue();
} }
float getParameter(const char *name) { float getParameter(const char *name) {
for (auto &parameter : parameters) { for (auto &parameter : parameters) {
if (strcmp(parameter.name, name) == 0) { if (strcmp(parameter.name, name) == 0) {
return *parameter.variable; return parameter.getValue();
} }
} }
return NAN; return NAN;
@@ -117,7 +122,8 @@ float getParameter(const char *name) {
bool setParameter(const char *name, const float value) { bool setParameter(const char *name, const float value) {
for (auto &parameter : parameters) { for (auto &parameter : parameters) {
if (strcmp(parameter.name, name) == 0) { if (strcmp(parameter.name, name) == 0) {
*parameter.variable = value; if (parameter.integer && !isfinite(value)) return false; // can't set integer to NaN or Inf
parameter.setValue(value);
return true; return true;
} }
} }
@@ -130,16 +136,18 @@ void syncParameters() {
if (motorsActive()) return; // don't use flash while flying, it may cause a delay if (motorsActive()) return; // don't use flash while flying, it may cause a delay
for (auto &parameter : parameters) { for (auto &parameter : parameters) {
if (parameter.value == *parameter.variable) continue; if (parameter.getValue() == parameter.cache) continue; // no change
if (isnan(parameter.value) && isnan(*parameter.variable)) continue; // handle NAN != NAN if (isnan(parameter.getValue()) && isnan(parameter.cache)) continue; // both are NaN
storage.putFloat(parameter.name, *parameter.variable); if (isinf(parameter.getValue()) && isinf(parameter.cache)) continue; // both are Inf
parameter.value = *parameter.variable;
storage.putFloat(parameter.name, parameter.getValue());
parameter.cache = parameter.getValue(); // update cache
} }
} }
void printParameters() { void printParameters() {
for (auto &parameter : parameters) { for (auto &parameter : parameters) {
print("%s = %g\n", parameter.name, *parameter.variable); print("%s = %g\n", parameter.name, parameter.getValue());
} }
} }