From f8de487c2f590d6f4e27f878ac104c229667702c Mon Sep 17 00:00:00 2001 From: Linus Torvalds Date: Sun, 4 Sep 2011 13:34:22 -0700 Subject: [PATCH] Make a guess at the cylinder description from the size and pressure I'll want to also add a way to override/set the cylinder type: both manually by just setting a size in liters, and by picking from some list of standard cylinder sizes. For example, it looks like most of my dives are marked as having 12-liter cylinders. That is probably some default from Suunto Dive Manager, or from whatever Dirk did. It's almost certainly not right for any of them: as far as I know, the standard cylinders for Lahaina Divers (which is likely most of the warm water dives) are AL72's for air, and AL80's for Nitrox. That would be a 10L and a 11.1L tank respectively, afaik. I don't know what a 12-liter tank would be or where that size comes from. Anyway, the LP85+ tank designation for some of the dives looks more likely: that's one of the common sizes I've used for local dives. So the size of that thing is much more probably correct. Signed-off-by: Linus Torvalds --- parse-xml.c | 69 +++++++++++++++++++++++++++++++++++++++++++++-------- save-xml.c | 3 +++ 2 files changed, 62 insertions(+), 10 deletions(-) diff --git a/parse-xml.c b/parse-xml.c index 3221111..1ca046d 100644 --- a/parse-xml.c +++ b/parse-xml.c @@ -708,6 +708,51 @@ static void sanitize_gasmix(gasmix_t *mix) memset(mix, 0, sizeof(*mix)); } +/* + * See if the size/workingpressure looks like some standard cylinder + * size, eg "AL80". + */ +static void match_standard_cylinder(cylinder_type_t *type) +{ + int psi, cuft, len; + const char *fmt; + char buffer[20], *p; + + /* Do we already have a cylinder description? */ + if (type->description) + return; + + cuft = type->size.mliter / 1000; + psi = type->workingpressure.mbar / 68.95; + + switch (psi) { + case 2300 ... 2500: /* 2400 psi: LP tank */ + fmt = "LP%d"; + break; + case 2600 ... 2700: /* 2640 psi: LP+10% */ + fmt = "LP%d+"; + break; + case 2900 ... 3100: /* 3000 psi: ALx tank */ + fmt = "AL%d"; + break; + case 3400 ... 3500: /* 3442 psi: HP tank */ + fmt = "HP%d"; + break; + case 3700 ... 3850: /* HP+10% */ + fmt = "HP%d+"; + break; + default: + return; + } + len = snprintf(buffer, sizeof(buffer), fmt, cuft); + p = malloc(len+1); + if (!p) + return; + memcpy(p, buffer, len+1); + type->description = p; +} + + /* * There are two ways to give cylinder size information: * - total amount of gas in cuft (depends on working pressure and physical size) @@ -720,20 +765,24 @@ static void sanitize_gasmix(gasmix_t *mix) */ static void sanitize_cylinder_type(cylinder_type_t *type) { + double volume_of_air, atm, volume; + /* If we have no working pressure, it had *better* be just a physical size! */ if (!type->workingpressure.mbar) return; - /* - * 35l tanks? Do they exist? - * Assume this is a "size in cuft" thing. - */ - if (type->size.mliter > 35000) { - double volume_of_air = type->size.mliter * 28.317; /* cu ft to milliliter */ - double atm = type->workingpressure.mbar / 1013.25; /* working pressure in atm */ - double volume = volume_of_air / atm; /* milliliters at 1 atm: "true size" */ - type->size.mliter = volume; - } + /* No size either? Nothing to go on */ + if (!type->size.mliter) + return; + + /* Ok, we have both size and pressure: try to match a description */ + match_standard_cylinder(type); + + /* .. and let's assume that the 'size' was cu ft of air */ + volume_of_air = type->size.mliter * 28.317; /* milli-cu ft to milliliter */ + atm = type->workingpressure.mbar / 1013.25; /* working pressure in atm */ + volume = volume_of_air / atm; /* milliliters at 1 atm: "true size" */ + type->size.mliter = volume + 0.5; } static void sanitize_cylinder_info(struct dive *dive) diff --git a/save-xml.c b/save-xml.c index c33cbb0..46bcec0 100644 --- a/save-xml.c +++ b/save-xml.c @@ -127,6 +127,7 @@ static void save_cylinder_info(FILE *f, struct dive *dive) for (i = 0; i < MAX_CYLINDERS; i++) { cylinder_t *cylinder = dive->cylinder+i; int volume = cylinder->type.size.mliter; + const char *description = cylinder->type.description; int o2 = cylinder->gasmix.o2.permille; int he = cylinder->gasmix.he.permille; @@ -141,6 +142,8 @@ static void save_cylinder_info(FILE *f, struct dive *dive) } if (volume) fprintf(f, " size='%u.%03u l'", FRACTION(volume, 1000)); + if (description) + fprintf(f, " name='%s'", description); fprintf(f, " />\n"); } } -- 2.45.2