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

Fix per_cylinder_mean_depth calculation for CCR

Browse source 
This patch is bigger than necessary because I also renamed the
get_cylinder_use() function to the much more accurate
get_cylinder_idx_by_use().

If we have no gas changes (except for a possible explicit first gas),
(which in the CCR case also means no bailout), this code will give you
correct per cylinder depth and duration for oxygen and diluent and
therefore create more reasonable gas consumption data for CCR dives.

Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
This commit is contained in:
Dirk Hohndel 2014-11-17 00:09:59 +00:00
parent ee7dd4307a
commit 3e5cb7e2aa
2 changed files with 17 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

24
dive.c
View file

@ -698,10 +698,18 @@ void per_cylinder_mean_depth(struct dive *dive, struct divecomputer *dc, int *me
for (i = 0; i < MAX_CYLINDERS; i++)
mean[i] = duration[i] = 0;
struct event *ev = get_next_event(dc->events, "gaschange");
if (!ev) {
// special case - no gas changes
mean[0] = dc->meandepth.mm;
duration[0] = dc->duration.seconds;
if (!ev || (dc && dc->sample && ev->time.seconds == dc->sample[0].time.seconds && get_next_event(ev->next, "gaschange") == NULL)) {
// we have either no gas change or only one gas change and that's setting an explicit first cylinder
if (dc->dctype == CCR) {
int o2_cyl = get_cylinder_idx_by_use(dive, OXYGEN);
int diluent_cyl = get_cylinder_idx_by_use(dive, DILUENT);
mean[o2_cyl] = mean[diluent_cyl] = dc->meandepth.mm;
duration[o2_cyl] = duration[diluent_cyl] = dc->duration.seconds;
} else {
// OC, no real gas changes, it's all the first cylinder
mean[explicit_first_cylinder(dive, dc)] = dc->meandepth.mm;
duration[explicit_first_cylinder(dive, dc)] = dc->duration.seconds;
}
return;
}
if (!dc->samples)
@ -1066,8 +1074,8 @@ unsigned int dc_airtemp(struct divecomputer *dc)
static void fixup_cylinder_use(struct dive *dive) // for CCR dives, store the indices
{ // of the oxygen and diluent cylinders
dive->oxygen_cylinder_index = get_cylinder_use(dive, OXYGEN);
dive->diluent_cylinder_index = get_cylinder_use(dive, DILUENT);
dive->oxygen_cylinder_index = get_cylinder_idx_by_use(dive, OXYGEN);
dive->diluent_cylinder_index = get_cylinder_idx_by_use(dive, DILUENT);
}
static void fixup_airtemp(struct dive *dive)
@ -1543,13 +1551,13 @@ static void merge_weightsystem_info(weightsystem_t *res, weightsystem_t *a, weig
*res = *a;
}
/* get_cylinder_use(): Find the index of the first cylinder with a particular CCR use type.
/* get_cylinder_idx_by_use(): Find the index of the first cylinder with a particular CCR use type.
* The index returned corresponds to that of the first cylinder with a cylinder_use that
* equals the appropriate enum value [oxygen, diluent, bailout] given by cylinder_use_type.
* A negative number returned indicates that a match could not be found.
* Call parameters: dive = the dive being processed
* cylinder_use_type = an enum, one of {oxygen, diluent, bailout} */
extern int get_cylinder_use(struct dive *dive, enum cylinderuse cylinder_use_type)
extern int get_cylinder_idx_by_use(struct dive *dive, enum cylinderuse cylinder_use_type)
{
int cylinder_index;
for (cylinder_index = 0; cylinder_index < MAX_CYLINDERS; cylinder_index++) {

2
dive.h
View file

@ -341,7 +341,7 @@ struct dive {
int oxygen_cylinder_index, diluent_cylinder_index; // CCR dive cylinder indices
};
extern int get_cylinder_use(struct dive *dive, enum cylinderuse cylinder_use_type);
extern int get_cylinder_idx_by_use(struct dive *dive, enum cylinderuse cylinder_use_type);
/* when selectively copying dive information, which parts should be copied? */
struct dive_components {