Skip to content
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

Signification modification to unittests for the water-atcor-develop branch #115

Merged

Conversation

Oceancolour-RG
Copy link

In this commit the following changes where made:

  1. The removal of the original test wagl-input dataset for the unittest;
  2. The addition of a new test wagl-input dataset consistent with wagl-water-atcor products;
  3. code cleanup and modification of down-sampling code to accommodate the mndwi.h5 file;
  4. modification of the appropriate unittests to accommodate the wagl-water-atcor products;
  5. black formatting

Rodrigo Garcia and others added 30 commits October 26, 2020 15:29
…Sentinel-2's absent level-1 metadata yaml file
omad and others added 26 commits November 12, 2020 17:36
Useful when it's potentially very expensive to do all the vector
operations to determine the valid region from the data itself.
The old dc_to_stac function is used externally so we need to keep
a shim of it for backwards compatibilty.
- Use one session for all downloads
- Use a timeout
- Keep cached items much longer when they're stable Stac versions.
- Allow skipping of the cache
- Allow no-network-access usage
- Verbose option for users
Cattrs removed 3.6 support, yet it's still trying
to install the new version.
@sixy6e sixy6e assigned sixy6e and unassigned sixy6e Nov 13, 2020
Copy link
Contributor

@sixy6e sixy6e left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Quite a lot of changes, but looks good to me.

@sixy6e sixy6e merged commit 9d8f984 into opendatacube:water-atcor-develop Nov 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants