X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;f=profile.c;h=7fc7850394f6c0312364cca5e390a5ff833375f8;hb=0d7ad02f87ad02d3b9836c07230f2ddb3f1d3e14;hp=ad158fe8a1f9aaafb40b792f31dea13667594f5f;hpb=edbba678b3cdc8f06a84a965b04f13ef1f4ecaab;p=ext%2Fsubsurface.git

diff --git a/profile.c b/profile.c
index ad158fe..7fc7850 100644
--- a/profile.c
+++ b/profile.c
@@ -96,6 +96,24 @@ void set_source_rgb(struct graphics_context *gc, double r, double g, double b)
 
 #define ROUND_UP(x,y) ((((x)+(y)-1)/(y))*(y))
 
+/* debugging tool - not normally used */
+static void dump_pi (struct plot_info *pi)
+{
+	int i;
+
+	printf("pi:{nr:%d maxtime:%d meandepth:%d maxdepth:%d \n"
+		"    minpressure:%d maxpressure:%d endpressure:%d mintemp:%d maxtemp:%d\n",
+		pi->nr, pi->maxtime, pi->meandepth, pi->maxdepth,
+		pi->minpressure, pi->maxpressure, pi->endpressure, pi->mintemp, pi->maxtemp);
+	for (i = 0; i < pi->nr; i++)
+		printf("    entry[%d]:{same_cylinder:%d cylinderindex:%d sec:%d pressure:{%d,%d}\n"
+			"                temperature:%d depth:%d smoothed:%d}\n",
+			i, pi->entry[i].same_cylinder, pi->entry[i].cylinderindex, pi->entry[i].sec,
+			pi->entry[i].pressure[0], pi->entry[i].pressure[1],
+			pi->entry[i].temperature, pi->entry[i].depth, pi->entry[i].smoothed);
+	printf("   }\n");
+}
+
 /*
  * When showing dive profiles, we scale things to the
  * current dive. However, we don't scale past less than
@@ -630,6 +648,8 @@ static void plot_pressure_value(struct graphics_context *gc, int mbar, int sec,
 	plot_text(gc, &tro, sec, mbar, "%d %s", pressure, unit);
 }
 
+#define GET_PRESSURE(_entry) (SENSOR_PRESSURE(_entry) ? : INTERPOLATED_PRESSURE(_entry))
+
 static void plot_cylinder_pressure_text(struct graphics_context *gc, struct plot_info *pi)
 {
 	int i;
@@ -642,14 +662,21 @@ static void plot_cylinder_pressure_text(struct graphics_context *gc, struct plot
 	if (!get_cylinder_pressure_range(gc, pi))
 		return;
 
-	/* only loop over the actual events from the dive computer */
-	for (i = 2; i < pi->nr; i++) {
+	/* only loop over the actual events from the dive computer
+	 * plus the second synthetic event at the start (to make sure
+	 * we get "time=0" right)
+	 * sadly with a recent change that first entry may no longer
+	 * have any pressure reading - in that case just grab the
+	 * pressure from the second entry */
+	if (GET_PRESSURE(pi->entry + 1) == 0 && GET_PRESSURE(pi->entry + 2) !=0)
+		INTERPOLATED_PRESSURE(pi->entry + 1) = GET_PRESSURE(pi->entry + 2);
+	for (i = 1; i < pi->nr; i++) {
 		entry = pi->entry + i;
 
 		if (!entry->same_cylinder) {
 			cyl = entry->cylinderindex;
 			if (!seen_cyl[cyl]) {
-				mbar = SENSOR_PRESSURE(entry) ? : INTERPOLATED_PRESSURE(entry);
+				mbar = GET_PRESSURE(entry);
 				plot_pressure_value(gc, mbar, entry->sec, LEFT, BOTTOM);
 				seen_cyl[cyl] = TRUE;
 			}
@@ -657,14 +684,13 @@ static void plot_cylinder_pressure_text(struct graphics_context *gc, struct plot
 				/* remember the last pressure and time of
 				 * the previous cylinder */
 				cyl = (entry - 1)->cylinderindex;
-				last_pressure[cyl] =
-					SENSOR_PRESSURE(entry - 1) ? : INTERPOLATED_PRESSURE(entry - 1);
+				last_pressure[cyl] = GET_PRESSURE(entry - 1);
 				last_time[cyl] = (entry - 1)->sec;
 			}
 		}
 	}
 	cyl = entry->cylinderindex;
-	last_pressure[cyl] = SENSOR_PRESSURE(entry) ? : INTERPOLATED_PRESSURE(entry);
+	last_pressure[cyl] = GET_PRESSURE(entry);
 	last_time[cyl] = entry->sec;
 
 	for (cyl = 0; cyl < MAX_CYLINDERS; cyl++) {
@@ -751,7 +777,7 @@ static struct plot_info *analyze_plot_info(struct plot_info *pi)
 	/* Do pressure min/max based on the non-surface data */
 	for (i = 0; i < nr; i++) {
 		struct plot_data *entry = pi->entry+i;
-		int pressure = SENSOR_PRESSURE(entry) ? : INTERPOLATED_PRESSURE(entry);
+		int pressure = GET_PRESSURE(entry);
 		int temperature = entry->temperature;
 
 		if (pressure) {
@@ -871,8 +897,9 @@ static void fill_missing_tank_pressures(struct dive *dive, struct plot_info *pi,
 		cur_pr[cyl] = track_pr[cyl]->start;
 	}
 
-	/* The first two are "fillers" */
-	for (i = 2; i < pi->nr; i++) {
+	/* The first two are "fillers", but in case we don't have a sample
+	 * at time 0 we need to process the second of them here */
+	for (i = 1; i < pi->nr; i++) {
 		entry = pi->entry + i;
 		if (SENSOR_PRESSURE(entry)) {
 			cur_pr[entry->cylinderindex] = SENSOR_PRESSURE(entry);
@@ -911,7 +938,8 @@ static void fill_missing_tank_pressures(struct dive *dive, struct plot_info *pi,
 				INTERPOLATED_PRESSURE(entry) =
 					cur_pr[entry->cylinderindex] + cur_pt * magic;
 				cur_pr[entry->cylinderindex] = INTERPOLATED_PRESSURE(entry);
-			}
+			} else
+				INTERPOLATED_PRESSURE(entry) = cur_pr[entry->cylinderindex];
 		}
 	}
 }
@@ -978,6 +1006,19 @@ static void check_gas_change_events(struct dive *dive, struct plot_info *pi)
 	set_cylinder_index(pi, i, cylinderindex, ~0u);
 }
 
+/* for computers that track gas changes through events */
+static int count_gas_change_events(struct dive *dive)
+{
+	int count = 0;
+	struct event *ev = get_next_gaschange(dive->events);
+
+	while (ev) {
+		count++;
+		ev = get_next_gaschange(ev->next);
+	}
+	return count;
+}
+
 /*
  * Create a plot-info with smoothing and ranged min/max
  *
@@ -989,48 +1030,103 @@ static struct plot_info *create_plot_info(struct dive *dive, int nr_samples, str
 {
 	int cylinderindex = -1;
 	int lastdepth, lastindex;
-	int i, nr = nr_samples + 4, sec, cyl;
-	size_t alloc_size = plot_info_size(nr);
+	int i, pi_idx, nr, sec, cyl;
+	size_t alloc_size;
 	struct plot_info *pi;
 	pr_track_t *track_pr[MAX_CYLINDERS] = {NULL, };
 	pr_track_t *pr_track, *current;
 	gboolean missing_pr = FALSE;
 	struct plot_data *entry = NULL;
+	struct event *ev;
 
+	/* we want to potentially add synthetic plot_info elements for the gas changes */
+	nr = nr_samples + 4 + 2 * count_gas_change_events(dive);
+	alloc_size = plot_info_size(nr);
 	pi = malloc(alloc_size);
 	if (!pi)
 		return pi;
 	memset(pi, 0, alloc_size);
 	pi->nr = nr;
+	pi_idx = 2; /* the two extra events at the start */
+	/* check for gas changes before the samples start */
+	ev = get_next_gaschange(dive->events);
+	while (ev && ev->time.seconds < dive_sample->time.seconds) {
+		entry = pi->entry + pi_idx;
+		entry->sec = ev->time.seconds;
+		entry->depth = 0; /* is that always correct ? */
+		pi_idx++;
+		ev = get_next_gaschange(ev->next);
+	}
+	if (ev && ev->time.seconds == dive_sample->time.seconds) {
+		/* we already have a sample at the time of the event */
+		ev = get_next_gaschange(ev->next);
+	}
 	sec = 0;
 	lastindex = 0;
 	lastdepth = -1;
 	for (i = 0; i < nr_samples; i++) {
 		int depth;
+		int delay = 0;
 		struct sample *sample = dive_sample+i;
 
-		entry = pi->entry + i + 2;
-		sec = entry->sec = sample->time.seconds;
+		entry = pi->entry + i + pi_idx;
+		while (ev && ev->time.seconds < sample->time.seconds) {
+			/* insert two fake plot info structures for the end of
+			 * the old tank and the start of the new tank */
+			entry->sec = ev->time.seconds;
+			(entry+1)->sec = ev->time.seconds + 1;
+			/* we need a fake depth - let's interpolate */
+			if (i) {
+				entry->depth = sample->depth.mm -
+					(sample->depth.mm - (sample-1)->depth.mm) / 2;
+			} else
+				entry->depth = sample->depth.mm;
+			(entry+1)->depth = entry->depth;
+			pi_idx += 2;
+			entry = pi->entry + i + pi_idx;
+			ev = get_next_gaschange(ev->next);
+		}
+		if (ev && ev->time.seconds == sample->time.seconds) {
+			/* we already have a sample at the time of the event
+			 * just add a new one for the old tank and delay the
+			 * real even by one second (to keep time monotonous) */
+			entry->sec = ev->time.seconds;
+			entry->depth = sample->depth.mm;
+			pi_idx++;
+			entry = pi->entry + i + pi_idx;
+			ev = get_next_gaschange(ev->next);
+			delay = 1;
+		}
+		sec = entry->sec = sample->time.seconds + delay;
 		depth = entry->depth = sample->depth.mm;
 		entry->cylinderindex = sample->cylinderindex;
 		SENSOR_PRESSURE(entry) = sample->cylinderpressure.mbar;
 		entry->temperature = sample->temperature.mkelvin;
 
 		if (depth || lastdepth)
-			lastindex = i+2;
+			lastindex = i + pi_idx;
 
 		lastdepth = depth;
 		if (depth > pi->maxdepth)
 			pi->maxdepth = depth;
 	}
-
+	entry = pi->entry + i + pi_idx;
+	/* are there still unprocessed gas changes? that would be very strange */
+	while (ev) {
+		entry->sec = ev->time.seconds;
+		entry->depth = 0; /* why are there gas changes after the dive is over? */
+		pi_idx++;
+		entry = pi->entry + i + pi_idx;
+		ev = get_next_gaschange(ev->next);
+	}
+	nr = nr_samples + pi_idx - 2;
 	check_gas_change_events(dive, pi);
 
 	for (cyl = 0; cyl < MAX_CYLINDERS; cyl++) /* initialize the start pressures */
 		track_pr[cyl] = pr_track_alloc(dive->cylinder[cyl].start.mbar, -1);
 	current = track_pr[pi->entry[2].cylinderindex];
-	for (i = 0; i < nr_samples; i++) {
-		entry = pi->entry + i + 2;
+	for (i = 0; i < nr + 1; i++) {
+		entry = pi->entry + i + 1;
 
 		entry->same_cylinder = entry->cylinderindex == cylinderindex;
 		cylinderindex = entry->cylinderindex;
@@ -1068,11 +1164,23 @@ static struct plot_info *create_plot_info(struct dive *dive, int nr_samples, str
 			pr_track->end = pr;
 		}
 	}
-	/* Fill in the last two entries with empty values but valid times */
-	i = nr_samples + 2;
+	/* Fill in the last two entries with empty values but valid times
+	 * without creating a false cylinder change event */
+	i = nr + 2;
 	pi->entry[i].sec = sec + 20;
+	pi->entry[i].same_cylinder = 1;
+	pi->entry[i].cylinderindex = pi->entry[i-1].cylinderindex;
+	INTERPOLATED_PRESSURE(pi->entry + i) = GET_PRESSURE(pi->entry + i - 1);
 	pi->entry[i+1].sec = sec + 40;
+	pi->entry[i+1].same_cylinder = 1;
+	pi->entry[i+1].cylinderindex = pi->entry[i-1].cylinderindex;
+	INTERPOLATED_PRESSURE(pi->entry + i + 1) = GET_PRESSURE(pi->entry + i - 1);
+	/* the number of actual entries - some computers have lots of
+	 * depth 0 samples at the end of a dive, we want to make sure
+	 * we have exactly one of them at the end */
 	pi->nr = lastindex+1;
+	while (pi->nr <= i+2 && pi->entry[pi->nr-1].depth > 0)
+		pi->nr++;
 	pi->maxtime = pi->entry[lastindex].sec;
 
 	pi->endpressure = pi->minpressure = dive->cylinder[0].end.mbar;
@@ -1085,6 +1193,8 @@ static struct plot_info *create_plot_info(struct dive *dive, int nr_samples, str
 	}
 	for (cyl = 0; cyl < MAX_CYLINDERS; cyl++)
 		list_free(track_pr[cyl]);
+	if (0) /* awesome for debugging - not useful otherwise */
+		dump_pi(pi);
 	return analyze_plot_info(pi);
 }