Hi there,
I see that both MacDive and DivingLog claim Wisdom 3 support... Looking at the dump from a Wisdom 3 its reply to the version command is "WISDOM R3B 512K" and the model at address 0x0008 is 0x4458 which we don't appear to have covered in src/oceanic_atom2_parser.c
Has anyone sat down and looked at which parser layout is correct for the Wisdom 3? We have a bug report where most everything seems to work, but the temperature readings during the dive are off. Given the insanity of how Oceanic stores temperatures... it seems entirely possible that they did yet another minor "tweak" there. But if someone else has worked this out already I'd love to just get this fixed in libdivecomputer :-)
Thanks
/D
On 27-10-14 19:41, Dirk Hohndel wrote:
I see that both MacDive and DivingLog claim Wisdom 3 support... Looking at the dump from a Wisdom 3 its reply to the version command is "WISDOM R3B 512K" and the model at address 0x0008 is 0x4458 which we don't appear to have covered in src/oceanic_atom2_parser.c
That means it's using the default. I'm sure you won't be surprised if I say that's not necessary correct :-)
Has anyone sat down and looked at which parser layout is correct for the Wisdom 3? We have a bug report where most everything seems to work, but the temperature readings during the dive are off. Given the insanity of how Oceanic stores temperatures... it seems entirely possible that they did yet another minor "tweak" there. But if someone else has worked this out already I'd love to just get this fixed in libdivecomputer :-)
I don't have any changes for the Wisdom 3 pending.
Jef
On Tue, Oct 28, 2014 at 10:11:59AM +0100, Jef Driesen wrote:
On 27-10-14 19:41, Dirk Hohndel wrote:
I see that both MacDive and DivingLog claim Wisdom 3 support... Looking at the dump from a Wisdom 3 its reply to the version command is "WISDOM R3B 512K" and the model at address 0x0008 is 0x4458 which we don't appear to have covered in src/oceanic_atom2_parser.c
That means it's using the default. I'm sure you won't be surprised if I say that's not necessary correct :-)
I was actually surprised how well it did.
In the mean time I got a bin file plus the dump of a decoded dive from the Sherwood website and it decoded almost everything correctly.
Only the sign of the temperature delta appears to be inverted. I asked the person who submitted the bug to verify my analysis, but I'm guessing you'll have a patch from me to fully enable the Wisdom 3 later today.
/D
On Tue, Oct 28, 2014 at 07:24:40AM -0700, Dirk Hohndel wrote:
On Tue, Oct 28, 2014 at 10:11:59AM +0100, Jef Driesen wrote:
On 27-10-14 19:41, Dirk Hohndel wrote:
I see that both MacDive and DivingLog claim Wisdom 3 support... Looking at the dump from a Wisdom 3 its reply to the version command is "WISDOM R3B 512K" and the model at address 0x0008 is 0x4458 which we don't appear to have covered in src/oceanic_atom2_parser.c
That means it's using the default. I'm sure you won't be surprised if I say that's not necessary correct :-)
I was actually surprised how well it did.
In the mean time I got a bin file plus the dump of a decoded dive from the Sherwood website and it decoded almost everything correctly.
Only the sign of the temperature delta appears to be inverted. I asked the person who submitted the bug to verify my analysis, but I'm guessing you'll have a patch from me to fully enable the Wisdom 3 later today.
And here we go. This appears to be all that's needed to fix the bug reported by the Subsurface user. I have not looked into any other possible differences to the Wisdom 2.
From the specs I can find those two seem pretty identical with the
exception of the deco algorithm. "modified Haldanean" on the Wisdom 2 and DSAT (Oceanics rather ill specified proprietary algorithm) on the Wisdom 3.
/D