GeoIPS algorithm plugin RGB_Default was added in #530 and seems completely unused. I'm not too sure why this was added, but nonetheless, the top-level docstring for that algorithm is incorrect and needs clarification. We need to update this accordingly.
Also, the product_defaults plugin RGB-Default doesn't adhere to proper naming definition. We should rename this to RGB_Default or rename algorithm RGB_Default to RGB-Default. Personally I think we should use the underscore nomenclature because that is largely used throughout GeoIPS.
Background and Motivation
PR #530 introduced these files and we should address them sooner rather than later, for the aforementioned reasons.
Requested Update
Description
GeoIPS algorithm plugin
RGB_Default
was added in #530 and seems completely unused. I'm not too sure why this was added, but nonetheless, the top-level docstring for that algorithm is incorrect and needs clarification. We need to update this accordingly.Also, the product_defaults plugin RGB-Default doesn't adhere to proper naming definition. We should rename this to RGB_Default or rename algorithm
RGB_Default
toRGB-Default
. Personally I think we should use the underscore nomenclature because that is largely used throughout GeoIPS.Background and Motivation
PR #530 introduced these files and we should address them sooner rather than later, for the aforementioned reasons.
Code to demonstrate issue
algorithms:RGB_Default
product_defaults:RGB-Default
products:viirs:Night-Vis-IR
products:viirs:Night-Vis-IR-GeoIPS1
Checklist for Completion
algorithms:RGB_Default
product_defaults:RGB-Default