Graph documentation [was: Re: Meaning of GF settings]

[Dirk Hohndel]

"Robert C. Helling" <helling at lmu.de> writes:

> On Fri, 11 Jan 2013, Dirk Hohndel wrote:
>
>> But all kidding aside, I'm a bit nervous about this one. I think I'd
>> rather have us use the math as it is defined and have the people who
>> want to be more aggressive do so on their own.
>
> We could have this as a configurable option (e.g. "acceptable ceiling 
> violation"), maybe not only Jan would be interested in this.
>
> As far as the planning goes as a compiler option the attached patch should 
> be sufficient. If it is of general interest to play with this, of course 
> it could/should be hooked up to the gui/preferences.

Can I get a Signed-off-by for this patch? Also, I think I'll have the
default be 0...

/D



> index c1b0fb4..97ae865 100644
> --- a/planner.c
> +++ b/planner.c
> @@ -12,6 +12,7 @@
>  #include "divelist.h"
>  #include "display-gtk.h"
>  
> +#define ACCEPTABLE_CEILING_VIOLATION 200
>  
>  int decostoplevels[] = { 0, 3000, 6000, 9000, 12000, 15000, 18000, 21000, 24000, 27000,
>  		     30000, 33000, 36000, 39000, 42000, 45000, 48000, 51000, 54000, 57000,
> @@ -125,7 +126,7 @@ int time_at_last_depth(struct dive *dive, int next_stop, char **cached_data_p)
>  	depth = sample->depth.mm;
>  	get_gas_from_events(&dive->dc, sample->time.seconds, &o2, &he);
>  	gasidx = get_gasidx(dive, o2, he);
> -	while (deco_allowed_depth(tissue_tolerance, surface_pressure, dive, 1) > next_stop) {
> +	while (deco_allowed_depth(tissue_tolerance, surface_pressure, dive, 1) > next_stop + ACCEPTABLE_CEILING_VIOLATION) {
>  		wait++;
>  		tissue_tolerance = add_segment(depth_to_mbar(depth, dive) / 1000.0,
>  					       &dive->cylinder[gasidx].gasmix, 1, sample->po2, dive);