ceos-seo / data_cube_ui

Data Cube user interface allowing users to interact with the Data Cube and run sample analysis cases
http://ceos-cube.org/
Apache License 2.0
47 stars 51 forks source link

Small Problem with data_cube_ui/apps/dc_algorithm/models/application_models.py #25

Open wscullen opened 6 years ago

wscullen commented 6 years ago

When trying to get the mosiac tool to not do a cloud based clean mask, I came across a problem in the get_clean_mask_func(): in apps/dc_algorithm/models/application_models.py, where if you were to not specify bit_mask or cf_mask it would crash because the return_all_true() has a problem where it is calling .shape on a numpy array as a function:

def return_all_true(ds):
     return np.full(ds[self.get_measurements()[0]].shape(), True)

Should be:

def return_all_true(ds):
     return np.full(ds[self.get_measurements()[0]].shape, True)

without the parenthesis after .shape. This isn't a bug that the mosaic tool would run into normally but I thought I would post it just in case anyone has this happen to them.