no dives are shown in the Dive List
Js
jun_song23 at yahoo.com
Mon Jan 27 15:53:21 UTC 2014
Morning guys (where i lived),
Appreciate you guys looking into this problem, i have a clue why this may happened.
I logged 64 dives while my ostc mk2 is on firmware v1.7, then i updated to firmware v2.8 and made 2 more dives. Using jDiveLog, it successfully downloaded the latest 2 dives while ignoring the rest on its own accord.
I tried to rectify by reflashing back to old firmware but not successful.
Regards,
Jun
Dirk Hohndel <dirk at hohndel.org> wrote:
>On Mon, 2014-01-27 at 20:26 +0100, Jef Driesen wrote:
>> On 27-01-14 19:10, Dirk Hohndel wrote:
>> > On Mon, 2014-01-27 at 10:53 +0100, Jef Driesen wrote:
>> >> I think the real problem is something else. Jun Song's dives fail to
>> >> parse. Not sure why yet. But that's just one part of the problem. When I
>> >> try download the memory dump into subsurface, I can confirm no dives are
>> >> imported. No error message is shown. So it seems subsurface is silently
>> >> dropping dives that are failing to parse. That's the second part of the
>> >> problem.
>> >>
>> >> I quickly checked the subsurface code. The dive_cb function exits
>> >> immediately in case of a parsing error, and as a result the dive never
>> >> gets added anywhere.
>> >
>> > Which seems reasonable. If we can't parse it, we clearly can't add it.
>>
>> Depends. If there is one small parsing error that does not necessary mean you
>> didn't get any useful info? It's not always black and white. I also realize this
>> is tricky.
>>
>> This is one of the reasons why I recommend keeping the raw data around. Because
>> then you can re-parse the dive again when the bug is fixed, without having to
>> download again. If it's a nasty bug that takes a while to fix (or you dive very
>> often) then by the time the bug is fixed, those dives may already have been
>> pushed out of the dive computer's memory. If you still have the raw data, that's
>> not a problem.
>
>A great point in theory. The amount of effort to keep the data around...
>I don't know. Doesn't seem worth it.
>
>> >> But the error is also never reported back to the
>> >> user. There is a call to the dev_info function to show an error message
>> >> in the progress bar, but I assume that because the download dialog is
>> >> closed almost immediately, the user doesn't have a chance to notice this
>> >> error message.
>> >
>> > I'll fix that.
>>
>> I think this will already be a great improvement. Silent data loss is nasty and
>> confusing.
>
>Try the latest master. Not beautiful, but effective, I think.
>
>
>/D
>
More information about the subsurface
mailing list