[Subsurface-divelog/subsurface] Improve OTU calculations (#1851)

[Willem Ferguson]

Hi Robert,
The new calculation using 6/5 is exactly double the OTU value that is 
computed with the Baker (=Clark) algorithm.
Change the 60 to 120?
Kind regards,
wf

> Argh, I had my maths wrong (6/5 instead of 5/6, which makes me think, 
> didn't the original literature have 1 .2 there?). OK, assuming 5/6 is 
> correct, have a look at 
> https://github.com/atdotde/subsurface/blob/c9d550e36fbbe4a3a10a6add808935c82c92efb6/core/divelist.c#L209 
> This formula seems to get it (sufficiently) right. Have a look at that 
> branch, that is not for merging but it has all the debugging info in 
> it. If you like it, you could take that formula, get rid of the printf 
> and your old formula (that is now with delta_old), and instead fo the 
> delta do out += immediately. Also in the if's further up, it's now 
> fine to have po2 = 500 since we nowhere divide by this anymore.
>
> Still need to write that blog post...
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub 
> <https://github.com/Subsurface-divelog/subsurface/pull/1851#discussion_r234420744>, 
> or mute the thread 
> <https://github.com/notifications/unsubscribe-auth/AE1-sn7aMs3IS9tfotLj2z4IFS5X37Auks5uwHUtgaJpZM4YYa-4>.
>


-- 
This message and attachments are subject to a disclaimer.

Please refer to 
http://upnet.up.ac.za/services/it/documentation/docs/004167.pdf 
<http://upnet.up.ac.za/services/it/documentation/docs/004167.pdf> for
full 
details.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.subsurface-divelog.org/pipermail/subsurface/attachments/20181118/cd014c1d/attachment.html>