Oh, nice. I was too careful (or should have checked this before writing my last email)... it turns out that most of Henrik's issues appear to be addressed with my changes...
On Thu, May 29, 2014 at 03:23:26PM +0200, Henrik Brautaset Aronsen wrote: > > Nice work everyone! I stumbled upon a couple of things: > > 1) As I planned a dive (10 mins @ 58 meters with 18/45 and 50%) I noticed > that the MOD, EAD and EADD was wrong: 57, 56 & 56m, while it should be > something like 68, 23 and 33. See > https://dl.dropboxusercontent.com/u/1087529/planner1.png We "fixed" this by removing the info overlay (actually I think it would be correct even if we didn't remove the info overlay - prior to my changes today the profile was rendered with the wrong gases). Serious question - did you find the info overlay useful? Robert and I felt that it was just in the way and so he disabled that... > 2) The background on that plan is red, which leads me to believe that > something is wrong? Maybe related to the wrong values in 1)? Yes, this was a side effect of the same bug - we were using the wrong gases when rendering the profile, so even though we correctly calculated the plan / deco, with the different gases that were used to render the profile, the dive violated the calculated ceiling and therefore turned red. > 3) When I save the dive, the values from 1) change. Still wrong: 18, 32 & > 60m. See https://dl.dropboxusercontent.com/u/1087529/planner2.png. The > ssrf file is here: https://dl.dropboxusercontent.com/u/1087529/planner.ssrf I now get (at 58m): MOD 67, END 27, EADD 23 Those seem correct to me (but are different from what you said you expected. > 4) 18/45 is nicely shown on the graph when I use that gas, but EAN50 isn't > shown after the gas switch. That's because the calculation for the used gas was completely bogus and that caused the rendering of the gas to fail. This seems to work now. > 5) I got a crash when planning a second dive, but I haven't been able to > reproduce it: https://dl.dropboxusercontent.com/u/1087529/plannercrash.txt This I can't reproduce. I do get a strange "too many gas mixes" warning when planning this dive. I'll need to look into that one... But overall I think we are making serious progress ;-) /D _______________________________________________ subsurface mailing list [email protected] http://lists.hohndel.org/cgi-bin/mailman/listinfo/subsurface
