Commit f946f8c7 authored by Don Gagne's avatar Don Gagne

Decimal place support from meta data

parent 114f00c5
...@@ -131,7 +131,22 @@ QVariant Fact::value(void) const ...@@ -131,7 +131,22 @@ QVariant Fact::value(void) const
QString Fact::valueString(void) const QString Fact::valueString(void) const
{ {
return _value.toString(); QString valueString;
switch (type()) {
case FactMetaData::valueTypeFloat:
qDebug() << name() << value() << decimalPlaces();
valueString = QString("%1").arg(value().toFloat(), 0, 'g', decimalPlaces());
break;
case FactMetaData::valueTypeDouble:
valueString = QString("%1").arg(value().toDouble(), 0, 'g', decimalPlaces());
break;
default:
valueString = value().toString();
break;
}
return valueString;
} }
QVariant Fact::defaultValue(void) const QVariant Fact::defaultValue(void) const
...@@ -222,6 +237,16 @@ bool Fact::maxIsDefaultForType(void) const ...@@ -222,6 +237,16 @@ bool Fact::maxIsDefaultForType(void) const
} }
} }
int Fact::decimalPlaces(void) const
{
if (_metaData) {
return _metaData->decimalPlaces();
} else {
qWarning() << "Meta data pointer missing";
return FactMetaData::defaultDecimalPlaces;
}
}
QString Fact::group(void) const QString Fact::group(void) const
{ {
if (_metaData) { if (_metaData) {
......
...@@ -46,22 +46,23 @@ public: ...@@ -46,22 +46,23 @@ public:
const Fact& operator=(const Fact& other); const Fact& operator=(const Fact& other);
Q_PROPERTY(int componentId READ componentId CONSTANT) Q_PROPERTY(int componentId READ componentId CONSTANT)
Q_PROPERTY(QString name READ name CONSTANT) Q_PROPERTY(QString group READ group CONSTANT)
Q_PROPERTY(QVariant value READ value WRITE setValue NOTIFY valueChanged USER true) Q_PROPERTY(QString name READ name CONSTANT)
Q_PROPERTY(QVariant valueString READ valueString NOTIFY valueChanged) Q_PROPERTY(QVariant value READ value WRITE setValue NOTIFY valueChanged USER true)
Q_PROPERTY(QString units READ units CONSTANT) Q_PROPERTY(QVariant valueString READ valueString NOTIFY valueChanged)
Q_PROPERTY(QVariant defaultValue READ defaultValue CONSTANT) Q_PROPERTY(QString units READ units CONSTANT)
Q_PROPERTY(bool defaultValueAvailable READ defaultValueAvailable CONSTANT) Q_PROPERTY(QVariant defaultValue READ defaultValue CONSTANT)
Q_PROPERTY(bool valueEqualsDefault READ valueEqualsDefault NOTIFY valueChanged) Q_PROPERTY(bool defaultValueAvailable READ defaultValueAvailable CONSTANT)
Q_PROPERTY(FactMetaData::ValueType_t type READ type CONSTANT) Q_PROPERTY(bool valueEqualsDefault READ valueEqualsDefault NOTIFY valueChanged)
Q_PROPERTY(QString shortDescription READ shortDescription CONSTANT) Q_PROPERTY(FactMetaData::ValueType_t type READ type CONSTANT)
Q_PROPERTY(QString longDescription READ longDescription CONSTANT) Q_PROPERTY(QString shortDescription READ shortDescription CONSTANT)
Q_PROPERTY(QVariant min READ min CONSTANT) Q_PROPERTY(QString longDescription READ longDescription CONSTANT)
Q_PROPERTY(bool minIsDefaultForType READ minIsDefaultForType CONSTANT) Q_PROPERTY(QVariant min READ min CONSTANT)
Q_PROPERTY(QVariant max READ max CONSTANT) Q_PROPERTY(bool minIsDefaultForType READ minIsDefaultForType CONSTANT)
Q_PROPERTY(bool maxIsDefaultForType READ maxIsDefaultForType CONSTANT) Q_PROPERTY(QVariant max READ max CONSTANT)
Q_PROPERTY(QString group READ group CONSTANT) Q_PROPERTY(bool maxIsDefaultForType READ maxIsDefaultForType CONSTANT)
Q_PROPERTY(int decimalPlaces READ decimalPlaces CONSTANT)
/// Convert and validate value /// Convert and validate value
/// @param convertOnly true: validate type conversion only, false: validate against meta data as well /// @param convertOnly true: validate type conversion only, false: validate against meta data as well
...@@ -86,6 +87,7 @@ public: ...@@ -86,6 +87,7 @@ public:
QVariant max(void) const; QVariant max(void) const;
bool maxIsDefaultForType(void) const; bool maxIsDefaultForType(void) const;
QString group(void) const; QString group(void) const;
int decimalPlaces(void) const;
/// Sets and sends new value to vehicle even if value is the same /// Sets and sends new value to vehicle even if value is the same
void forceSetValue(const QVariant& value); void forceSetValue(const QVariant& value);
......
...@@ -32,30 +32,32 @@ ...@@ -32,30 +32,32 @@
#include <limits> #include <limits>
FactMetaData::FactMetaData(QObject* parent) : FactMetaData::FactMetaData(QObject* parent)
QObject(parent), : QObject(parent)
_group("*Default Group"), , _group("*Default Group")
_type(valueTypeInt32), , _type(valueTypeInt32)
_defaultValue(0), , _defaultValue(0)
_defaultValueAvailable(false), , _defaultValueAvailable(false)
_min(_minForType()), , _min(_minForType())
_max(_maxForType()), , _max(_maxForType())
_minIsDefaultForType(true), , _minIsDefaultForType(true)
_maxIsDefaultForType(true) , _maxIsDefaultForType(true)
, _decimalPlaces(defaultDecimalPlaces)
{ {
} }
FactMetaData::FactMetaData(ValueType_t type, QObject* parent) : FactMetaData::FactMetaData(ValueType_t type, QObject* parent)
QObject(parent), : QObject(parent)
_group("*Default Group"), , _group("*Default Group")
_type(type), , _type(type)
_defaultValue(0), , _defaultValue(0)
_defaultValueAvailable(false), , _defaultValueAvailable(false)
_min(_minForType()), , _min(_minForType())
_max(_maxForType()), , _max(_maxForType())
_minIsDefaultForType(true), , _minIsDefaultForType(true)
_maxIsDefaultForType(true) , _maxIsDefaultForType(true)
, _decimalPlaces(defaultDecimalPlaces)
{ {
} }
...@@ -76,6 +78,7 @@ const FactMetaData& FactMetaData::operator=(const FactMetaData& other) ...@@ -76,6 +78,7 @@ const FactMetaData& FactMetaData::operator=(const FactMetaData& other)
_max = other._max; _max = other._max;
_minIsDefaultForType = other._minIsDefaultForType; _minIsDefaultForType = other._minIsDefaultForType;
_maxIsDefaultForType = other._maxIsDefaultForType; _maxIsDefaultForType = other._maxIsDefaultForType;
_decimalPlaces = other._decimalPlaces;
return *this; return *this;
} }
......
...@@ -59,18 +59,19 @@ public: ...@@ -59,18 +59,19 @@ public:
const FactMetaData& operator=(const FactMetaData& other); const FactMetaData& operator=(const FactMetaData& other);
// Property accessors // Property accessors
QString name(void) const { return _name; } QString name(void) const { return _name; }
QString group(void) const { return _group; } QString group(void) const { return _group; }
ValueType_t type(void) const { return _type; } ValueType_t type(void) const { return _type; }
QVariant defaultValue(void) const; QVariant defaultValue(void) const;
bool defaultValueAvailable(void) const { return _defaultValueAvailable; } bool defaultValueAvailable(void) const { return _defaultValueAvailable; }
QString shortDescription(void) const { return _shortDescription; } QString shortDescription(void) const { return _shortDescription; }
QString longDescription(void) const { return _longDescription;} QString longDescription(void) const { return _longDescription;}
QString units(void) const { return _units; } QString units(void) const { return _units; }
QVariant min(void) const { return _min; } QVariant min(void) const { return _min; }
QVariant max(void) const { return _max; } QVariant max(void) const { return _max; }
bool minIsDefaultForType(void) const { return _minIsDefaultForType; } bool minIsDefaultForType(void) const { return _minIsDefaultForType; }
bool maxIsDefaultForType(void) const { return _maxIsDefaultForType; } bool maxIsDefaultForType(void) const { return _maxIsDefaultForType; }
int decimalPlaces(void) const { return _decimalPlaces; }
// Property setters // Property setters
void setName(const QString& name) { _name = name; } void setName(const QString& name) { _name = name; }
...@@ -81,6 +82,7 @@ public: ...@@ -81,6 +82,7 @@ public:
void setUnits(const QString& units) { _units = units; } void setUnits(const QString& units) { _units = units; }
void setMin(const QVariant& max); void setMin(const QVariant& max);
void setMax(const QVariant& max); void setMax(const QVariant& max);
void setDecimalPlaces(int decimalPlaces) { _decimalPlaces = decimalPlaces; }
/// Converts the specified value, validating against meta data /// Converts the specified value, validating against meta data
/// @param value Value to convert, can be string /// @param value Value to convert, can be string
...@@ -90,6 +92,8 @@ public: ...@@ -90,6 +92,8 @@ public:
/// @returns false: Convert failed, errorString set /// @returns false: Convert failed, errorString set
bool convertAndValidate(const QVariant& value, bool convertOnly, QVariant& typedValue, QString& errorString); bool convertAndValidate(const QVariant& value, bool convertOnly, QVariant& typedValue, QString& errorString);
static const int defaultDecimalPlaces = 3;
private: private:
QVariant _minForType(void) const; QVariant _minForType(void) const;
QVariant _maxForType(void) const; QVariant _maxForType(void) const;
...@@ -106,6 +110,7 @@ private: ...@@ -106,6 +110,7 @@ private:
QVariant _max; QVariant _max;
bool _minIsDefaultForType; bool _minIsDefaultForType;
bool _maxIsDefaultForType; bool _maxIsDefaultForType;
int _decimalPlaces;
}; };
#endif #endif
...@@ -299,6 +299,19 @@ void PX4ParameterLoader::loadParameterFactMetaData(void) ...@@ -299,6 +299,19 @@ void PX4ParameterLoader::loadParameterFactMetaData(void)
qCDebug(PX4ParameterLoaderLog) << "Unit:" << text; qCDebug(PX4ParameterLoaderLog) << "Unit:" << text;
metaData->setUnits(text); metaData->setUnits(text);
} else if (elementName == "decimal") {
Q_ASSERT(metaData);
QString text = xml.readElementText();
qCDebug(PX4ParameterLoaderLog) << "Decimal:" << text;
bool convertOk;
QVariant varDecimals = QVariant(text).toUInt(&convertOk);
if (convertOk) {
metaData->setDecimalPlaces(varDecimals.toInt());
} else {
qCWarning(PX4ParameterLoaderLog) << "Invalid decimals value, name:" << metaData->name() << " type:" << metaData->type() << " decimals:" << text << " error: invalid number";
}
} else { } else {
qDebug() << "Unknown element in XML: " << elementName; qDebug() << "Unknown element in XML: " << elementName;
} }
......
...@@ -20,15 +20,6 @@ This file is part of the QGROUNDCONTROL project ...@@ -20,15 +20,6 @@ This file is part of the QGROUNDCONTROL project
======================================================================*/ ======================================================================*/
/**
* @file
* @brief MissionItem class
*
* @author Benjamin Knecht <mavteam@student.ethz.ch>
* @author Petri Tanskanen <mavteam@student.ethz.ch>
*
*/
#include <QStringList> #include <QStringList>
#include <QDebug> #include <QDebug>
...@@ -115,10 +106,12 @@ MissionItem::MissionItem(QObject* parent, ...@@ -115,10 +106,12 @@ MissionItem::MissionItem(QObject* parent,
FactMetaData* latitudeMetaData = new FactMetaData(FactMetaData::valueTypeDouble, _latitudeFact); FactMetaData* latitudeMetaData = new FactMetaData(FactMetaData::valueTypeDouble, _latitudeFact);
latitudeMetaData->setUnits("deg"); latitudeMetaData->setUnits("deg");
latitudeMetaData->setDecimalPlaces(7);
FactMetaData* longitudeMetaData = new FactMetaData(FactMetaData::valueTypeDouble, _longitudeFact); FactMetaData* longitudeMetaData = new FactMetaData(FactMetaData::valueTypeDouble, _longitudeFact);
longitudeMetaData->setUnits("deg"); longitudeMetaData->setUnits("deg");
longitudeMetaData->setDecimalPlaces(7);
FactMetaData* altitudeMetaData = new FactMetaData(FactMetaData::valueTypeDouble, _altitudeFact); FactMetaData* altitudeMetaData = new FactMetaData(FactMetaData::valueTypeDouble, _altitudeFact);
altitudeMetaData->setUnits("meters"); altitudeMetaData->setUnits("meters");
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment