1
0
mirror of https://github.com/tumic0/GPXSee.git synced 2024-12-05 00:39:09 +01:00

Compare commits

...

2 Commits

Author SHA1 Message Date
aec052edaf Fixed elevation source selection logic 2023-04-10 17:21:02 +02:00
1bbc57173e Add only valid graph segments to the graphs
Fixes #489
2023-04-10 13:06:19 +02:00
11 changed files with 59 additions and 66 deletions

View File

@ -47,7 +47,7 @@ QList<GraphItem*> CadenceGraph::loadData(const Data &data)
const Track &track = data.tracks().at(i);
const Graph &graph = track.cadence();
if (!graph.isValid()) {
if (graph.isEmpty()) {
_palette.nextColor();
graphs.append(0);
} else {

View File

@ -85,7 +85,7 @@ void ElevationGraph::setInfo()
GraphItem *ElevationGraph::loadGraph(const Graph &graph, PathType type,
const QColor &color, bool primary)
{
if (!graph.isValid())
if (graph.isEmpty())
return 0;
ElevationGraphItem *gi = new ElevationGraphItem(graph, _graphType, _width,

View File

@ -50,7 +50,7 @@ QList<GraphItem*> GearRatioGraph::loadData(const Data &data)
for (int i = 0; i < data.tracks().count(); i++) {
const Graph &graph = data.tracks().at(i).ratio();
if (!graph.isValid()) {
if (graph.isEmpty()) {
_palette.nextColor();
graphs.append(0);
} else {

View File

@ -8,8 +8,6 @@ GraphItem::GraphItem(const Graph &graph, GraphType type, int width,
const QColor &color, Qt::PenStyle style, QGraphicsItem *parent)
: GraphicsItem(parent), _graph(graph), _type(type), _secondaryGraph(0)
{
Q_ASSERT(_graph.isValid());
_units = Metric;
_sx = 0; _sy = 0;
_color = color;

View File

@ -47,7 +47,7 @@ QList<GraphItem*> HeartRateGraph::loadData(const Data &data)
const Track &track = data.tracks().at(i);
const Graph &graph = track.heartRate();
if (!graph.isValid()) {
if (graph.isEmpty()) {
_palette.nextColor();
graphs.append(0);
} else {

View File

@ -47,7 +47,7 @@ QList<GraphItem*> PowerGraph::loadData(const Data &data)
const Track &track = data.tracks().at(i);
const Graph &graph = track.power();
if (!graph.isValid()) {
if (graph.isEmpty()) {
_palette.nextColor();
graphs.append(0);
} else {

View File

@ -50,7 +50,7 @@ void SpeedGraph::setInfo()
GraphItem *SpeedGraph::loadGraph(const Graph &graph, const Track &track,
const QColor &color, bool primary)
{
if (!graph.isValid())
if (graph.isEmpty())
return 0;
SpeedGraphItem *gi = new SpeedGraphItem(graph, _graphType, _width,

View File

@ -51,7 +51,7 @@ QList<GraphItem*> TemperatureGraph::loadData(const Data &data)
const Track &track = data.tracks().at(i);
const Graph &graph = track.temperature();
if (!graph.isValid()) {
if (graph.isEmpty()) {
_palette.nextColor();
graphs.append(0);
} else {

View File

@ -46,16 +46,6 @@ typedef QVector<GraphPoint> GraphSegment;
class Graph : public QList<GraphSegment>
{
public:
bool isValid() const
{
if (isEmpty())
return false;
for (int i = 0; i < size(); i++)
if (at(i).size() < 2)
return false;
return true;
}
bool hasTime() const
{
for (int i = 0; i < size(); i++) {

View File

@ -68,16 +68,14 @@ GraphPair Route::elevation() const
{
if (_useDEM) {
Graph dem(demElevation());
if (dem.isValid())
return GraphPair(dem, _show2ndElevation ? gpsElevation() : Graph());
else
return GraphPair(gpsElevation(), Graph());
return (dem.isEmpty())
? GraphPair(gpsElevation(), Graph())
: GraphPair(dem, _show2ndElevation ? gpsElevation() : Graph());
} else {
Graph gps(gpsElevation());
if (gps.isValid())
return GraphPair(gps, _show2ndElevation ? demElevation() : Graph());
else
return GraphPair(demElevation(), Graph());
return (gps.isEmpty())
? GraphPair(demElevation(), Graph())
: GraphPair(gps, _show2ndElevation ? demElevation() : Graph());
}
}

View File

@ -266,7 +266,8 @@ Graph Track::gpsElevation() const
sd.at(j).elevation()));
}
ret.append(filter(gs, _elevationWindow));
if (gs.size() >= 2)
ret.append(filter(gs, _elevationWindow));
}
if (_data.style().color().isValid())
@ -293,7 +294,8 @@ Graph Track::demElevation() const
gs.append(GraphPoint(seg.distance.at(j), seg.time.at(j), dem));
}
ret.append(filter(gs, _elevationWindow));
if (gs.size() >= 2)
ret.append(filter(gs, _elevationWindow));
}
if (_data.style().color().isValid())
@ -306,16 +308,14 @@ GraphPair Track::elevation() const
{
if (_useDEM) {
Graph dem(demElevation());
if (dem.isValid())
return GraphPair(dem, _show2ndElevation ? gpsElevation() : Graph());
else
return GraphPair(gpsElevation(), Graph());
return (dem.isEmpty())
? GraphPair(gpsElevation(), Graph())
: GraphPair(dem, _show2ndElevation ? gpsElevation() : Graph());
} else {
Graph gps(gpsElevation());
if (gps.isValid())
return GraphPair(gps, _show2ndElevation ? demElevation() : Graph());
else
return GraphPair(demElevation(), Graph());
return (gps.isEmpty())
? GraphPair(demElevation(), Graph())
: GraphPair(gps, _show2ndElevation ? demElevation() : Graph());
}
}
@ -344,11 +344,13 @@ Graph Track::computedSpeed() const
gs.append(GraphPoint(seg.distance.at(j), seg.time.at(j), v));
}
ret.append(filter(gs, _speedWindow));
GraphSegment &filtered = ret.last();
if (gs.size() >= 2) {
ret.append(filter(gs, _speedWindow));
GraphSegment &filtered = ret.last();
for (int j = 0; j < stop.size(); j++)
filtered[stop.at(j)].setY(0);
for (int j = 0; j < stop.size(); j++)
filtered[stop.at(j)].setY(0);
}
}
if (_data.style().color().isValid())
@ -382,11 +384,13 @@ Graph Track::reportedSpeed() const
gs.append(GraphPoint(seg.distance.at(j), seg.time.at(j), v));
}
ret.append(filter(gs, _speedWindow));
GraphSegment &filtered = ret.last();
if (gs.size() >= 2) {
ret.append(filter(gs, _speedWindow));
GraphSegment &filtered = ret.last();
for (int j = 0; j < stop.size(); j++)
filtered[stop.at(j)].setY(0);
for (int j = 0; j < stop.size(); j++)
filtered[stop.at(j)].setY(0);
}
}
if (_data.style().color().isValid())
@ -399,18 +403,14 @@ GraphPair Track::speed() const
{
if (_useReportedSpeed) {
Graph reported(reportedSpeed());
if (reported.isValid())
return GraphPair(reported, _show2ndSpeed ? computedSpeed()
: Graph());
else
return GraphPair(computedSpeed(), Graph());
return (reported.isEmpty())
? GraphPair(computedSpeed(), Graph())
: GraphPair(reported, _show2ndSpeed ? computedSpeed() : Graph());
} else {
Graph computed(computedSpeed());
if (computed.isValid())
return GraphPair(computed, _show2ndSpeed ? reportedSpeed()
: Graph());
else
return GraphPair(reportedSpeed(), Graph());
return (computed.isEmpty())
? GraphPair(reportedSpeed(), Graph())
: GraphPair(computed, _show2ndSpeed ? reportedSpeed() : Graph());
}
}
@ -430,7 +430,8 @@ Graph Track::heartRate() const
gs.append(GraphPoint(seg.distance.at(j), seg.time.at(j),
sd.at(j).heartRate()));
ret.append(filter(gs, _heartRateWindow));
if (gs.size() >= 2)
ret.append(filter(gs, _heartRateWindow));
}
if (_data.style().color().isValid())
@ -456,7 +457,8 @@ Graph Track::temperature() const
sd.at(j).temperature()));
}
ret.append(gs);
if (gs.size() >= 2)
ret.append(gs);
}
if (_data.style().color().isValid())
@ -481,7 +483,8 @@ Graph Track::ratio() const
gs.append(GraphPoint(seg.distance.at(j), seg.time.at(j),
sd.at(j).ratio()));
ret.append(gs);
if (gs.size() >= 2)
ret.append(gs);
}
if (_data.style().color().isValid())
@ -515,11 +518,13 @@ Graph Track::cadence() const
gs.append(GraphPoint(seg.distance.at(j), seg.time.at(j), c));
}
ret.append(filter(gs, _cadenceWindow));
GraphSegment &filtered = ret.last();
if (gs.size() >= 2) {
ret.append(filter(gs, _cadenceWindow));
GraphSegment &filtered = ret.last();
for (int j = 0; j < stop.size(); j++)
filtered[stop.at(j)].setY(0);
for (int j = 0; j < stop.size(); j++)
filtered[stop.at(j)].setY(0);
}
}
if (_data.style().color().isValid())
@ -554,11 +559,13 @@ Graph Track::power() const
gs.append(GraphPoint(seg.distance.at(j), seg.time.at(j), p));
}
ret.append(filter(gs, _powerWindow));
GraphSegment &filtered = ret.last();
if (gs.size() >= 2) {
ret.append(filter(gs, _powerWindow));
GraphSegment &filtered = ret.last();
for (int j = 0; j < stop.size(); j++)
filtered[stop.at(j)].setY(0);
for (int j = 0; j < stop.size(); j++)
filtered[stop.at(j)].setY(0);
}
}
if (_data.style().color().isValid())