Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Problem loading an already processed image with Processing/Open picture file... #1

Open
perdrix52 opened this issue Mar 18, 2018 · 8 comments

Comments

@perdrix52
Copy link
Member

If use Processing/Load a Picture File, It takes a VERY long time to complete (many minutes), The code seems to spend most of its time in CDeepStack::ComputeOriginalHistogram().

Once the image has completed loading the image display is not correct, and the histogram is also not correct:
screen shot 1

If I open the same file using Registering and Stacking/Open picture files, it displays correctly.

screen shot 2

@tonk777
Copy link
Collaborator

tonk777 commented Apr 9, 2018

Isn't the second image shown also modified by the gamma slider to its top right? Move that slider about an see the image gamma change. So what you see is happens to be where ever the gamma slider is set.

@perdrix52
Copy link
Member Author

Yes it is but the bottom image shown is with the default gamma and is shown as it appears in other stuff like PS or GIMP. The upper one shown is when loading into the postprocessing and a gain up is applied prior to display which isn't necessarily wanted ...

@tonk777
Copy link
Collaborator

tonk777 commented Apr 9, 2018

I expect Luc can give the underlying reason and work flow. From experience this is what I have observed

After after initial stacking there is no stretch yet applied so most stacked images written out look black with a few stars - the data is still linear. This is how the image should look when loaded into the post processing window - black with a few brighter stars showing up. The gamma slider provides a display only stretch so you can see something of your stacked image. (in fact over the years on CN the most frequent new user issue is that they see a reasonable on-screen display but when they open the resultant written out file in something else they see near black. The response is always - "you have linear data - that is normal and expected - you need to do your own non-linear stretch in an application of your choice"

So the issue to sort out is - when is the DSS display really reflecting the content of the file and when its it applying a display only stretch of some sort as an visualization aid.


[[My own thoughts are that the DSS post processing tools are very weak and if used compromise useful post processing in other image processing tools (compresses dynamic range in the wrong places and this cant be recovered). This can be improved. There are at least two standard astrophotography non-linear transformations that could be incorporated into DSS that would improve this situation - DDP and ASINH - the former models chemical film processing and development while avoiding luminance saturation and latter similarly stretches but preserves the original colour balance. These would at least give the user the right leg up from linear to non-linear processing and deliver an image that looks right on both DSS's display and in other programs]]

@tonk777
Copy link
Collaborator

tonk777 commented Apr 9, 2018

This is just a rough idea - could we not use mouse rollover to show the real untreated image (mouse off) and the temporary gamma treated for visual aid version (mouse over)? That or an explicit toggle button.

@perdrix52
Copy link
Member Author

perdrix52 commented Apr 9, 2018

I just think that to start with the post processing screen should not apply any gain until asked to. FWIW I believe that Luc is somewhat in agreement on this.

As for other post processing tools did you not notice my suggestion on Slack to incorporte the processing engine from FITS Liberator which does ASinH and a slew of others in a readily understandable way.

Dave

@tonk777
Copy link
Collaborator

tonk777 commented Apr 10, 2018

Why not simplify this even further and just have DSS launch FITS liberator with the final stacked image to do the post processing steps - saves a lot or work and there is a familiar UI to use.

@tonk777
Copy link
Collaborator

tonk777 commented Apr 10, 2018

Actually that might not be a good idea re the FITSLiberator app. FITSLiberator is open source on GitHub and unfortunately its not been touched since 2011 - it looks dead in most respects. The last commit is a tweak to the README 3 years ago and prior to that the NSIS installer script was tweaked 6 years ago. The code itself is frozen for 7 years.

I like the idea of using the engine for various post processing things - but if we take this approach we should fork what we need into the DSS code base and maintain what we use for ourselves - no-one else is going to fix it in FITSLiberator land. I've grabbed the code to a local place and going to get it building in VS2017

@tonk777
Copy link
Collaborator

tonk777 commented Apr 10, 2018

I just think that to start with the post processing screen should not apply 
any gain until asked to. FWIW I believe that Luc is somewhat in 
agreement on this.

Which was my view too - as I said the biggest complaint from first time CN users of DSS over the last 10 years or so was the file you saved when opened elsewhere never looked like the image shown on the DSS display (due to the temp contrast tweak) and it was (correctly) too dark to see anything. Once they understood that they had to do a stretch themselves things settled down.

perdrix52 pushed a commit that referenced this issue Jun 10, 2019
Merge Head Repository To My Fork
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants