<html><head><meta http-equiv="content-type" content="text/html; charset=utf-8"></head><body dir="auto"><div>Hi Robert</div><div>Before I comment point by point let me mention that is is exploratory and that I do not have all the answers yet.</div><div>Yes I am a photographer and my intent is to have the images available regardless off the computer I am at.</div><div>Hence a Dropbox location would do the job. My expectation is that people are not wildly let ssrf manage tons of images unless they provide the space.</div><div>Long term I have some really cool features in mind.</div><div><br><div>Regards,<div>Guido</div><div>+41 79 3217739</div></div></div><div><br>Am 13.10.2015 um 11:49 schrieb Robert Helling <<a href="mailto:helling@atdotde.de">helling@atdotde.de</a>>:<br><br></div><blockquote type="cite"><div><meta http-equiv="Content-Type" content="text/html charset=utf-8">Hi Guido,<div class=""><br class=""><div><blockquote type="cite" class=""><div class="">On 13.10.2015, at 11:18, Guido Lerch <<a href="mailto:guido.lerch@gmail.com" class="">guido.lerch@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><span id="cid:6E20E588-6F74-49EA-A0F6-9B7AC1829A28@theorie.physik.uni-muenchen.de"><0005-Adding-image-management-fixing-stupid-bug.txt></span></div></blockquote></div><div class=""><br class=""></div><div class="">here are a few comments on this series of patches (not really in order).</div><div class=""><br class=""></div><div class="">Re this last one: Who tells you that paths never exceed 256 bytes? I would not be so sure about this, in particular given that they can be unicode (Dirk has this test user on Windows who’s name I forget but its some Mörtlí çßråù something but longer). Also not all OSes use / as a path separator (but if you use Qt for file/path operations it gets translated).</div></div></div></blockquote>Have that fixes already but not submitted based on a note from this group.<br><blockquote type="cite"><div><div class=""><div class=""><br class=""></div><div class="">In fact, I could not find the function that does the actual copying. Are you sure your patches are complete?</div></div></div></blockquote>It works on Mac, I am reusing a copy function that was existing already that does the translation for the copy process<br><blockquote type="cite"><div><div class=""><div class=""><br class=""></div><div class="">Maybe it also helps to explain the rationale behind the image hashing logic but first I should say, I am not a photographer myself so my experience with all this is in fact limited to test cases.</div></div></div></blockquote>Bummer, I missed this and will fix.<div>The copy is done correctly but the file names will not be stored unless picture handles this, I need to check.<br><blockquote type="cite"><div><div class=""><div class=""><br class=""></div><div class="">But I would imagine, if you are really into photography, your image library can easily be several GB, so I wanted to avoid doubling this “copying images into subsurface” (whatever that might mean) but rather use them in place. </div></div></div></blockquote>In place is tricky too as with the cloud service people might use the app from different computers.<br><blockquote type="cite"><div><div class=""><div class="">This also gives the advantage that you can still edit them (as I understand underwater photos all need heavy color and lighting adjustments to be usable) and they would still show up in the latest version in subsurface. (These considerations are of course already invalidated by the fact that the git storage format also stores the images and thus in a sense already solved the problem you are trying to solve).</div><div class=""><br class=""></div><div class="">Of course, one wants to have (at least) potential access to the images on all computers where you run subsurface. And Dropbox (and the like) is indeed a very convenient mechanism for that (which I use as well). But I think, this should be solved in sufficient generality and in particular it should not break when on different machines the Dropbox folders have different paths (for example I have machines where I am robert, I am helling on others and on some others I am even Robert.Helling, let alone different OSes where the path structure is entirely different). I had though if this could be attacked by doing some path translation or symlinking but in the end I decided (for me, your milage might differ) that not attempting that but displaying the images wherever they are is the right approach. And this is done by hashing. There, the user is completely free where in the file system the images can be found, as long as from time to time she points subsurface to a sub-tree that contains images so subsurface can learn the files in that subtree.</div><div class=""><br class=""></div><div class="">Yes, this does not do the copying but I think the current solution is more general than the one you are implementing. Currently, with your workflow, you could import the images from where they are into subsurface and then, if you want copy them (with finder or any other tool) to the dropbox and then on all computers run “Find moved images” and point that to the Dropbox and all your images will appear.</div></div></div></blockquote>I found this rather confusing, that's why I try something different.</div><div>Overall your input is valid and will finally lead into something that the user loves.</div><div>Btw the reason I made this configurable is that people that don't like it can use it the "old" way.<br><blockquote type="cite"><div><div class=""><div class=""><br class=""></div><div class="">I agree, that this can be confusing (as the user has to understand what is actually happening) and deserves a better UI but I am not good at coming up with such things.</div></div></div></blockquote>I keep on playing with this, maybe we'll find something that makes ssrf stronger.</div><div>I would not want to use it the current way as only a fraction of my pictures make it into the ssrf dive. The vast majorit sits on a hD that is not accessible from all my computers.<br><blockquote type="cite"><div><div class=""><div class=""><br class=""></div><div class="">Just my two cents.</div><div class=""><br class=""></div><div class="">Robert</div><div apple-content-edited="true" class="">
<div style="color: rgb(0, 0, 0); font-family: Menlo; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div style="color: rgb(0, 0, 0); font-family: Menlo; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div style="color: rgb(0, 0, 0); font-family: Menlo; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div style="color: rgb(0, 0, 0); font-family: Menlo; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><br class="">-- <br class="">.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oO<br class="">Robert C. Helling Elite Master Course Theoretical and Mathematical Physics<br class=""> Scientific Coordinator<br class=""> Ludwig Maximilians Universitaet Muenchen, Dept. Physik<br class=""> Phone: +49 89 2180-4523 Theresienstr. 39, rm. B339<br class=""> <a href="http://www.atdotde.de" class="">http://www.atdotde.de</a></div><div style="color: rgb(0, 0, 0); font-family: Menlo; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><br class=""></div><div style="color: rgb(0, 0, 0); font-family: Menlo; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">Enhance your privacy, use cryptography! My PGP keys have fingerprints</div><div style="color: rgb(0, 0, 0); font-family: Menlo; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">A9D1 A01D 13A5 31FA 6515 BB44 0820 367C 36BC 0C1D and</div><div style="color: rgb(0, 0, 0); font-family: Menlo; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">DCED 37B6 251C 7861 270D 5613 95C7 9D32 9A8D 9B8F</div><div style="color: rgb(0, 0, 0); font-family: Menlo; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><br class=""></div></div></div><br class="Apple-interchange-newline"></div><br class="Apple-interchange-newline"><br class="Apple-interchange-newline">
</div>
<br class=""></div></div></blockquote></div></body></html>