
<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">Hey Guido,<div class=""><br class=""></div><div class="">For a number of good reasons, our mailing list doesn't point back to itself when</div><div class="">responding. This means one CAN reply privately if one wants to, but one needs</div><div class="">to do Reply-All to keep the mailing list in the loop. I'd prefer to keep most discussions</div><div class="">on the mailing list unless there is a strong reason not to...</div><div class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Sep 8, 2015, at 5:48 AM, Guido Lerch <<a href="mailto:guido.lerch@gmail.com" class="">guido.lerch@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class="">Could you let me know how I can delete all existing dive sites, they seem not to be stored in<div class="">my logbook. Please see below.<br class=""></div></div></div></blockquote><div><br class=""></div><div>Interesting question. I don't see a real use case for normal users to want to do that.</div><div>So adding this as a UI feature seems weird. You can manually remove them from</div><div>an XML file - but that's a bit tedious as well (it's easy enough to delete them from</div><div>the start of the file - but I haven't checked what Subsurface does when it encounters</div><div>references to dive sites in the dives and then doesn't have those sites... so I think</div><div>one would have to edit each dive as well... but a little perl or awk script could do</div><div>that quite easily.</div><br class=""><blockquote type="cite" class=""><div class=""><div dir="ltr" class=""><div class=""><div class="gmail_extra"><div class="gmail_quote">2015-09-08 2:25 GMT+02:00 Dirk Hohndel <span dir="ltr" class=""><<a href="mailto:dirk@hohndel.org" target="_blank" class="">dirk@hohndel.org</a>></span>:<br class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Mon, Sep 07, 2015 at 11:49:02PM +0200, Guido Lerch wrote:<br class="">

> Hi Dirk<br class="">

><br class="">

> I looked in the dive site stuff ... comments below.<br class="">

><br class="">

> Another thought I had is that the interactive menu on the dive table should<br class="">

> contain a<br class="">

> menu to actually update a dive - what do you think ?<br class="">

<br class="">

</span>Not sure what you mean by that - can you explain in more detail what you<br class="">

have in mind?<br class=""></blockquote><div class=""><br class=""></div><div class="">it would be cool, in the future, if one could select one or multiple dives and say update</div><div class="">which would synchronize the Uemis with SubSurface if you for example modified an</div><div class="">entry on the computer or in SubSurface. </div></div></div></div></div></div></blockquote><div><br class=""></div><div>Again, I don't see the broad appeal to have a specialized UI for this. But you can</div><div>do this today. Go to the download menu and pick "download all" and "prefer divecomputer"</div><div>and then only pick the ones that you want to really get before accepting them. Admittedly,</div><div>especially on a Uemis with the incredibly slow and painful download this is not a good</div><div>experience.</div><div><br class=""></div><div>So if you have a good, solid use case why this is something people would really want</div><div>to do, we could consider adding a UI that makes this easier...</div><br class=""><blockquote type="cite" class=""><div class=""><div dir="ltr" class=""><div class=""><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<span class="">

> > This means that your algorithm creating dive sites in the Uemis downloader<br class="">

> > doesn't check<br class="">

> > if a dive site already exists before creating it.<br class="">

><br class="">

> true but I dont know (yet) how to do this, the way you coded that in is<br class="">

> that if it maps a dive detail you create<br class="">

> a new site (every time) called "from Uemis". At the end of your algorithm<br class="">

> you map and mark those dive site,<br class="">

> hence at the time you create a new one I am not sure how to check if a dive<br class="">

> site exists already.<br class="">

> I will look further into it but if you have a hint this would be<br class="">

> appreciated.<br class="">

<br class="">

</span>Yeah, I mentioned this in my first review. Don't use time(NULL) - this has<br class="">

second resolution and will therefore frequently return the same value for<br class="">

consecutive calls. Instead just use the dive->when which should be<br class="">

different for every dive that you have. This will give you a different<br class="">

uuid.<br class=""></blockquote><div class=""><br class=""></div><div class="">I have changed that but ended up with 1032 dive sites in Subsurface which makes</div><div class="">tracking down my current issues very difficult, I would like to start at zero with the </div><div class="">dive sites and see if the duplicate error's I am getting is going away with the new </div><div class="">code that I am testing currently. </div></div></div></div></div></div></blockquote><div><br class=""></div>Ah, that explains it. Yeah, that's a pain. And there is no trivial solution. This</div><div>shouldn't happen to anyone but developers (I hope), but it is very annoying.</div><div><br class=""></div><div>So one thing you can do to test your new code is of course to start with an</div><div>empty dive log - then you see if your algorithm now gets it right. That still</div><div>leaves you with the mess in your current file.</div><div><br class=""></div><div>Let me look for an easy way to deal with this. I'm thinking that just deleting</div><div>them from the beginning of the XML file should be enough. But I want to</div><div>make sure Subsurface does the right thing there.</div><div><br class=""></div></div><div>/D</div></body></html>