Page 1 of 1

Mixed Imperial/Metric Dives Upload Incorrectly from Perdix

Posted: Mon Jan 01, 2018 6:48 am
by Bollar
This seems like an edge case, so I don't know if you will want to bother to fix.

In between uploads to MacDive, and after an imperial dive, I changed my settings on the Shearwater Perdix AI to report depths in meters instead of feet. On uploading to MacDive iOS (which was still set to Imperial) the imperial dive depth of the newly uploaded dive was multiplied by three. I was able to work around this by deleting the new metric dives on the Perdix, importing the imperial dives and then undeleting the metric dives and importing them.

The dives uploaded correctly into Shearwater Cloud, so it seems like the data is available if you choose to address.

Re: Mixed Imperial/Metric Dives Upload Incorrectly from Perd

Posted: Mon Jan 01, 2018 7:01 am
by nick
Thanks for the report. I think I know what’s wrong here. I suspect the max depth is wrong but the actual samples are correct. I’ll check it out when I get home.

Re: Mixed Imperial/Metric Dives Upload Incorrectly from Perd

Posted: Mon Jan 01, 2018 9:44 am
by nick
Hmm. The code appears to be correct - the units are read and things converted as they should be. Can you clarify exactly which depth is incorrect - the max, or per-sample depths?

Also which version of MacDive for iOS specifically you are using.

Re: Mixed Imperial/Metric Dives Upload Incorrectly from Perd

Posted: Mon Jan 01, 2018 2:42 pm
by Bollar
nick wrote:Hmm. The code appears to be correct - the units are read and things converted as they should be. Can you clarify exactly which depth is incorrect - the max, or per-sample depths?

Also which version of MacDive for iOS specifically you are using.
It does appear that Max Depth is the incorrect value. The raw data on the Mac looks like reasonable foot numbers and the dive profile on iOS also looks reasonable.

This is iOS 1.2.7

Re: Mixed Imperial/Metric Dives Upload Incorrectly from Perd

Posted: Mon Jan 01, 2018 3:42 pm
by nick
Ahhhhh I found it. Thanks. Yup, total typo. It is indeed supposed to work, but there's a tiny mistake in there.

Edit: This is fixed, in both macOS and iOS. Unfortunately I just submitted a new build to Apple for review, but I could add you to TestFlight. I can send you a macOS beta if you need it.