[Patch] Fix planner notes gas change output logic
Joakim Bygdell
j.bygdell at gmail.com
Wed Jun 17 22:24:16 PDT 2015
>>
>>
>> Transition to 30 m in 1:40 min - runtime 1:40 on EAN32
>> Stay at 30 m for 28:20 min - runtime 30:00 on EAN32
>> Switch gas to air
>> Stay at 30 m for 30:00 min - runtime 60:00 on air
>> Switch gas to EAN32
>> Stay at 30 m for 30:00 min - runtime 90:00 on EAN32
>> Switch gas to EAN80
>> Stay at 9.0 m for 6:00 min - runtime 99:39 on EAN80
>> Stay at 6.0 m for 9:00 min - runtime 109:39 on EAN80
>> Stay at 3.0 m for 17:00 min - runtime 128:12 on EAN80
>> Transition to 0.0 m in 1:33 min - runtime 129:45 on EAN80
>>
>
> I think there is no problem with the verbatim plan.
That looks fine.
>
>
>>
>> depth
>> duration
>> runtime
>> gas
>> 30m
>> 2min
>> 2min
>> EAN32
>> 30m
>> 28min
>> 30min
>>
>> 30m
>> 30min
>> 60min
>> EAN32
>> 30m
>> 30min
>> 90min
>>
>> 9m
>> 10min
>> 100min
>> 6m
>> 10min
>> 110min
>> EAN80
>> 3m
>> 19min
>> 128min
>>
>> 0m
>> 2min
>> 130min
>>
>>
>
> This is wrong on many levels: Displaying EAN32 twice is of course nonsense, the second segment should be air and EAN80 should be printed on the 9m line, not at 6m.
The deep segments should look like this:
depth
duration
runtime
gas
30m
2min
2min
EAN32
30m
27min
29min
30m
1min
30min
air
30m
29min
59min
30m
1min
60min
EAN32
30m
29min
89min
I had to insert one minute segments where the gas switch happens for it to be displayed correctly.
Interestingly if I build the entire plan up until 9m including the stop times and gas switches it still gets it wrong.
depth
duration
runtime
gas
30m
2min
2min
EAN32
30m
27min
29min
30m
1min
30min
air
30m
29min
59min
30m
1min
60min
EAN32
30m
29min
89min
15m
2min
91min
9m
3min
94min
EAN80
9m
1min
95min
9m
4min
99min
6m
28min
127min
0m
3min
130min
Gas switch to EAN80 is specified during the 1min event at runtime 95min, now it gets printed to early.
/Jocke
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.subsurface-divelog.org/pipermail/subsurface/attachments/20150618/ac48d9da/attachment-0001.html>
More information about the subsurface
mailing list