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

multi-date handler aggregator in ramp styles can now receive raw data #1033

Merged
merged 5 commits into from
Jul 16, 2024

Conversation

SpacemanPaul
Copy link
Contributor

@SpacemanPaul SpacemanPaul commented Jul 15, 2024

Fixes #1007

In a multi-date handler for colour-ramp-type styles: a new pass_raw_data config entry controls what data is passed to the aggregator function:

pass_raw_data == False (the default):
The aggregator function is passed a time-dimensioned DataArray containing the output
of the index function.

pass_raw_data == True:
The aggregator function is passed a time-dimensioned Dataset, containing all of the
declared needed bands.


📚 Documentation preview 📚: https://datacube-ows--1033.org.readthedocs.build/en/1033/

@SpacemanPaul SpacemanPaul marked this pull request as ready for review July 15, 2024 23:09
Copy link
Contributor

@robbibt robbibt left a comment

Choose a reason for hiding this comment

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

This looks great @SpacemanPaul - the example in ows_test_cfgf.py should be a great example for me to use as a template for DEA Intertidal. Will do further testing once this is live in dev OWS

@SpacemanPaul SpacemanPaul merged commit 41daa21 into master Jul 16, 2024
11 checks passed
@SpacemanPaul SpacemanPaul deleted the ramp_mdh_raw_data branch July 16, 2024 02:07
SpacemanPaul added a commit that referenced this pull request Jul 31, 2024
…#1033)

* First attempt to implement raw data passing.

* Fix intermediate array dtype for numpy2

* Basic test.

* Wordlist update.

* Wordlist update.

(cherry picked from commit 41daa21)
SpacemanPaul added a commit that referenced this pull request Aug 1, 2024
* multi-date handler aggregator in ramp styles can now receive raw data (#1033)

* First attempt to implement raw data passing.

* Fix intermediate array dtype for numpy2

* Basic test.

* Wordlist update.

* Wordlist update.

(cherry picked from commit 41daa21)

* Prep for 1.8.41 release. (#1034)

* Prep for 1.8.41 release.

* Add this PR to HISTORY.rst

(cherry picked from commit 6c75bb2)

* Dockerfile: fix casing warning (#1035)

This fixes the warning:

- FromAsCasing: 'as' and 'FROM' keywords' casing do not match (line 2)

emitted by recent Docker releases.

(cherry picked from commit 955c49d)

* Version and top level folder (#1036)

* Make 'datacube-ows --version' work.

* Autowrap top level in single folder if necessary.

* Document top-level folder autowrapping behaviour.

* Update integration test translation files.

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Pass auth token to codecov GHA.

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

(cherry picked from commit d71b0ef)

* Changes to materialised view definition to prevent errors on databases with WKT CRSs (#1037)

* Changes to materialised view definition to prevent errors on databases with WKT CRSs.

* Make 'epsg:' checks case insensitive.

(cherry picked from commit fb88ea2)

* Port of #1039 - custom feature info enhancement.

* Resolve 1.8-1.9 cherry-picking conflicts.

* Appease the type checking gods.

* Remove unused import.

* Oops - dropped a line merging ows_configuration.py

---------

Co-authored-by: Peter A. Jonsson <[email protected]>
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.

Feature request: Allow aggregator function to receive raw band data instead of index function output
3 participants