Problem with VPM-B in master

[Rick Walsh]

Hi,

VPM-B is producing different plans in the latest master.  I have tracked it
down to commit 72806e42bc23ab7c7f2a639a1efee017388b53bc Replace global
in_planner variable by helper function.  I don't believe this commit was
supposed to change the resulting plans.

Also, the commit broke VPM-B conservatism.

Reverting the commit fixes the problem, but there's probably a better way
to fix it.

Using the 100 m, 10 min test plan:
Current master:

depth

duration

runtime

gas

100m

1min

1min

(18/45)

100m

9min

10min



33m

8min

18min



30m

2min

20min



27m

2min

22min



24m

3min

25min



21m

2min

27min

EAN50

18m

2min

29min



15m

3min

32min



12m

4min

36min



9m

6min

42min



6m

8min

50min

oxygen

3m

14min

64min



0m

0min

64min




The differences are more stark with repetitive dive plans.

Cheers,

Rick
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.subsurface-divelog.org/pipermail/subsurface/attachments/20150820/a58715de/attachment-0001.html>