-
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
New compositors: MultiFiller and LongitudeMaskingCompositor #1577
New compositors: MultiFiller and LongitudeMaskingCompositor #1577
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1577 +/- ##
==========================================
- Coverage 92.18% 91.10% -1.08%
==========================================
Files 251 251
Lines 36777 36865 +88
==========================================
- Hits 33902 33586 -316
- Misses 2875 3279 +404
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
Correct me if I'm wrong but couldn't the MultiFiller be replaced by using the Filler compositor on each of the inputs to an RGB?
Just curious, what do you use the LongitudeFiller for. I know you said for global composites but in what way? Would it make sense to also allow for latitude? |
DeepCode failed to analyze this pull requestSomething went wrong despite trying multiple times, sorry about that. |
The FillingCompositor fills each of the channels individually. Like a background for all bands. The MultiFiller is more like a chain of the existing Filler compositor. There's only one output. It can also use more than 3 projectables. In my case, I have images of 5 satellites. I'll attach a sample composite yaml in Monday. |
Oh I see now. In the past we've used the MultiScene's |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. I understand the need for the longitude masking now. It was just odd to see it in a compositor.
I've added to more compositors. They are necessary to create a "world composite" (global mosaic) as described in PR #1560 .
The
MultiFiller
is an extension ofFiller
but works with multiple projectables (including data from optional_datasets). Maybe Filler might be replaced by this one, but I am unsure if it has any side effects.The
LongitudeMaskingCompositor
makes it possible to mask areas outside spezified min and max longitude values.