mifr/subsurface
1
0
Fork 0
mirror of https://github.com/subsurface/subsurface.git synced 2025-02-19 22:16:15 +00:00
Code Issues Projects Releases Packages Wiki Activity

Use actual min and max temperatures in statistics.

Browse source 
The statistics page only used each dive's "watertemp" attribute,
regardless of actual higher/lower temperatures in the samples.  By
finding the actual max/min temperatures, the statistics page utilize
more "real" data, and look better even on single dives.

Signed-off-by: Henrik Brautaset Aronsen <subsurface@henrik.synth.no>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
This commit is contained in:
Henrik Brautaset Aronsen 2013-01-24 19:58:59 +01:00 • committed by Dirk Hohndel
parent b8efe709a8
commit e3088930ab
3 changed files with 47 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

41
statistics.c
View file

@ -107,6 +107,37 @@ enum {
static char * get_time_string(int seconds, int maxdays);
static void process_temperatures(struct dive *dp, stats_t *stats, const char *unit)
{
int min_temp, mean_temp, max_temp = 0;
if (dp->maxtemp.mkelvin)
max_temp = dp->maxtemp.mkelvin;
else
max_temp = dp->dc.watertemp.mkelvin;
if (max_temp && (!stats->max_temp || max_temp > stats->max_temp))
stats->max_temp = max_temp;
if (dp->mintemp.mkelvin)
min_temp = dp->mintemp.mkelvin;
else
min_temp = dp->dc.watertemp.mkelvin;
if (min_temp && (!stats->min_temp || min_temp < stats->min_temp))
stats->min_temp = min_temp;
if (min_temp || max_temp) {
mean_temp = min_temp;
if (mean_temp)
mean_temp = (mean_temp + max_temp) / 2;
else
mean_temp = max_temp;
stats->combined_temp += get_temp_units(mean_temp, &unit);
stats->combined_count++;
}
}
static void process_dive(struct dive *dp, stats_t *stats)
{
int old_tt, sac_time = 0;
@ -122,14 +153,8 @@ static void process_dive(struct dive *dp, stats_t *stats)
stats->max_depth.mm = dp->dc.maxdepth.mm;
if (stats->min_depth.mm == 0 || dp->dc.maxdepth.mm < stats->min_depth.mm)
stats->min_depth.mm = dp->dc.maxdepth.mm;
if (dp->dc.watertemp.mkelvin) {
if (stats->min_temp == 0 || dp->dc.watertemp.mkelvin < stats->min_temp)
stats->min_temp = dp->dc.watertemp.mkelvin;
if (dp->dc.watertemp.mkelvin > stats->max_temp)
stats->max_temp = dp->dc.watertemp.mkelvin;
stats->combined_temp += get_temp_units(dp->dc.watertemp.mkelvin, &unit);
stats->combined_count++;
}
process_temperatures(dp, stats, unit);
/* Maybe we should drop zero-duration dives */
if (!dp->dc.duration.seconds)