<div dir="ltr">Hi Akarsh,<div><br></div><div>I agree completely, I was also wondering why we were taking two steps for image download. I don't even think we should have "Save/Delete" all images or "save" for a single image, all images should be saved by default, let's reduce the GUI clutter here. Maybe we can add a button somewhere else for "Reset to Factory settings" or something like that where all images/dbs/settings are deleted in the ~/.local/share/kstars</div><div><br></div><div>Regards,</div><div>Jasem</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jan 8, 2016 at 2:17 AM, Akarsh Simha <span dir="ltr"><<a href="mailto:akarshsimha@gmail.com" target="_blank">akarshsimha@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Jasem<br>
<span class=""><br>
> So in observation planner, some images won't show up, I _think_ the problem<br>
> is here:<br>
><br>
><br>
> CurrentImagePath = QStandardPaths::locate( QStandardPaths::DataLocation ,<br>
> CurrentImage );<br>
><br>
> I think it should be :<br>
><br>
> CurrentImagePath =<br>
> QStandardPaths::writableLocation(QStandardPaths::DataLocation) +<br>
> QDir::separator() + CurrentImage<br>
<br>
</span>I'm not sure, because the former syntax seems to be achieving the same<br>
thing. But if it works, it works.<br>
<span class=""><br>
> Another thing, instead of hardcoding '/' where it could cause problems on<br>
> other operating systems, we should opt for QDir::separator()<br>
<br>
</span>I see, I was not sure what needed to be done.<br>
<br>
In fact, Jasem, I think that the entire point of having two kinds of<br>
images (Temporary an 'saved') is overkill and counterintuitive. I know<br>
we designed it with some purpose, but the way it works now, the "Temp"<br>
images are anyway stored in ~/.local/share/kstars -- so in what sense<br>
are they temp, I have no clue. So this entire thing is so<br>
counterintuitive to a developer, I don't know what it might be for a<br>
user. So I'm tempted to say that the entire image handling needs a<br>
rewrite. And while we are rewriting it, it might be useful to add<br>
support for multiple reference images, but not sure.<br>
<span class=""><br>
> Finally, if the downloaded image is large (> 10 MB) then the image is not<br>
> displayed, is there a way to limit the size of the image downloaded?<br>
<br>
</span>No idea here, other than trying the usual QFile::size() or something<br>
after download and backtracking.<br>
<br>
Regards<br>
<span class="HOEnZb"><font color="#888888">Akarsh<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div>Best Regards,<br>Jasem Mutlaq<br></div><div><br></div></div></div>
</div>