Helioviewer-Project / helioviewer.org

Main web application for solar data visualization
Other
18 stars 15 forks source link

Movies and screenshots do not respect layering order on screen. #257

Closed wafels closed 5 years ago

wafels commented 5 years ago

The LASCO C2/C3 layering order in the viewport is not respected in movies and screenshots. This is a screengrab:

Screen Shot 2019-03-16 at 5 02 02 PM

and this is the download generated by helioviewer.org

2019_03_09_19_45_29_LASCO_C3__LASCO_C2

wafels commented 5 years ago

The layering in the viewport should be respected in the movies/screenshots.

Helioviewer-Kirill commented 5 years ago

After some digging, this is not a bug and is by design in the codebase. Layering order is set based on a pre-defined list of instruments.

As it stands, the api will always render SOHO LASCO C3 over SOHO LASCO C2 and STEREO_A/B SECCHI COR2 over STEREO_A/B SECCHI COR1.

My assumption is that this preset layering order was implemented to create a properly layered image. However, it does not respect the layering order, on screen, as seen in the front-end.

This can be disabled. Let me know how I should proceed.

wafels commented 5 years ago

I think the layering order on screen should be respected. It's easier to understand how helioviewer works that way.

Helioviewer-Kirill commented 5 years ago

Fixed

wafels commented 5 years ago

Confirmed