Skip to content
This repository has been archived by the owner on Jun 18, 2023. It is now read-only.

CCDCesque: Use list of values for min_rmse #75

Closed
valpasq opened this issue Dec 9, 2015 · 1 comment
Closed

CCDCesque: Use list of values for min_rmse #75

valpasq opened this issue Dec 9, 2015 · 1 comment
Milestone

Comments

@valpasq
Copy link
Collaborator

valpasq commented Dec 9, 2015

We might expect different bands to vary in their RMSE due to differences in scaling. Therefore,
min_rmse hyperparameter for CCDCesque should accept either (a) a single value that will be applied to all bands (currently implemented) or (b) a list of values for each band, similar to how min_values/max_values for bands is currently structured.

@ceholden ceholden changed the title Use list of values for min_RMSE CCDCesque: Use list of values for min_rmse Dec 9, 2015
@ceholden ceholden added this to the v0.6.0 milestone Dec 9, 2015
ceholden added a commit that referenced this issue Dec 21, 2015
@ceholden
Copy link
Owner

Implemented suggestion in last commit. Given an example of using 7 (no-mask) bands, parameterize as follows:

min_rmse: 100

or

min_rmse: [100, 100, 100, 100, 100, 100, 100]

Seems to work as using an array of 7 "100" values passes the same test as using min_rmse=100.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants