-
Notifications
You must be signed in to change notification settings - Fork 300
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
Add missing EUMeTrain RGBs for SEVIRI and FCI #2670
Comments
I'd forget the HRV composites for FCI, there's no panchromatic channel. If one really wants to play with it, averaging the |
Thank you for this analysis! Regarding the new FCI composites: at EUM we have indeed a few recipes that we have been experimenting with in the last months for the various public releases. As discussed in Slack, we will try to push them to satpy during the upcoming PCW. I would say that as long as FCI is pre-operational, we are still quite free to change things (radically) without worrying too much. After it's operational we can think of a warning system for less-established RGBs, although the process for fine-tuning could go on for years :) |
For FCI IR Sandwich I've used |
Inevitably, FCI composites would differ from SEVIRI composites that use HRV. We would need to wait for a community consensus on how those definitions will be. I don't agree that we need to entirely forget about those composites, though; new composites can be developed that include comparable information at a 500-metre spatial resolution level. There's nothing that stops us with Pytroll/Satpy on proposing possible ways forward and marking those ways as experimental. Yes, for new FCI composites we can change everything all the time, but with the analysis I noticed that even for SEVIRI we don't always match community definitions, and that's a lot more dicey to change. |
Note on the IR sandwich: here we might want to resist the EUMeTrain standard and convince them to change. The EUMeTrain sandwhich colorises the colder part of the range using the rainbow colormap, whereas satpy applies spectral. We might want to bring this up with EUMeTrain. I'm not sure everyone is even aware satpy is doing something different. |
After #2780, the main missing point to close this issue are the missing tropical recipes for some RGBs. Considering that the current recipes basically cover the midlatitude cases, we could either
I would probably vote for the second since it's the clearest long-term, even though it introduces some duplication until we have a safe mechanism to deprecate composite (names).. |
For naming tropical or midlatitude variants, we might want to find a solution to #2644 first. |
I agree that the changes proposed in #2644 are very useful, but they would also require a new way of querying composites for users, which will take time to implement and to gain acceptance... I would be in favour of completing the list by adding the tropical recipes to the mix now, and then hopefully have a better query mechanism soon to improve the overall situation. |
I agree for the tropical recipes. Not sure about adding |
Feature Request
Is your feature request related to a problem? Please describe.
I would like to be able to generate all the "classic" EUMeTrain composites (11 RGBs and the Sandwich product) using both SEVIRI and FCI. Currently, one RGB (24-hour microphysics) is missing entirely, four composites are missing for FCI, and three have non-matching implementations.
Satpy is increasingly popular and when Satpy implements an RGB as a composite in a certain way, it may well become the de facto standard. At the same time, RGB communities from CIRA, NOAA, EUMETSAT, EUMeTrain, JMA, and others, are getting together in workshops to agree on standardised composite definitions. Where such a consensus exists, it would be desirable if this exact definition would work out-of-the-box in Satpy.
I went through the EUMeTrain RGBs to compare the Satpy implementation with the EUMeTrain definitions and describe what is absent or present, and whether definitions match between Satpy and EUMeTrain:
Describe the solution you'd like
I would like composite definitions for all EUMeTrain products that work for both SEVIRI and FCI.
Describe any changes to existing user workflow
Missing composites can be added, but diverging definitions would be difficult to change without causing problems for operational production. Where definitions differ, we may have to resort to adding a new composite with the tag
emt_
prepended. I don't think we should change the existing composites before we introduce breaking changes with Satpy 1.0, whenever that may be.Additional context
Satpy is increasingly popular to the degree that when Satpy gives an RGB/composite a certain name, this might become the de-facto standard. Therefore, it is important that when the RGB community does reach a consensus, that Satpy should consider implementing this.
See also:
Definitions used by other agencies may differ from the ones used by EUMeTrain.
Of course, I could make all those implementations locally, but I think the standard ones are worth having in Satpy with definitions that match the standard.
I have not considered the new composites that are possible with FCI, but not with SEVIRI, because I don't know if there is a community consensus yet on how they should be defined: True Colour RGB, Geocolor, Cloud Phase RGB, Cloud Type RGB, Fire Temperature RGB, or yet-to-be-named RGBs such as ones that include the solar moisture transmittance (SMT) using the 0.91 µm/0.86 µm ratio. Those should also be implemented in Satpy, but probably with a warning that definitions may change as community consensus on the definition develops.
The text was updated successfully, but these errors were encountered: