-
Notifications
You must be signed in to change notification settings - Fork 17
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
load_stac
scale and offset clarifications
#512
base: master
Are you sure you want to change the base?
Conversation
@m-mohr what should I add to make the
|
From the dev telco: Applies to both load_collection and load_stac.
Not sure whether we should add:
|
Instead of "manually" I would express this in terms of "client side" or "explicitly in the process graph", |
I agree, I would leave this out. There might be reasons that some kind of scaling is necessary, but it can not be expressed in structured metadata (e.g. because the scaling factors change at a certain date, or the scaling is non-linear) |
scale
andoffset
parameters, which from this discussion we aggreed shouldn't be applied by default: load_stac: scale_and_offset #503