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

Update opendatacube.md #76

Merged
merged 9 commits into from
Jun 13, 2024
Merged

Update opendatacube.md #76

merged 9 commits into from
Jun 13, 2024

Conversation

ValentinaHutter
Copy link
Contributor

Update to point to openeo-processes-dask and openeo-pg-parser-networkx.

add load_collection section with odc and stac
opendatacube, xarray and openeo-processes-python are **not** dependencies because this package simply creates a
python file that can be executed in the correct environment where these dependencies are resolved.
:::
* `opendatacube Products`: The implementation of `load_collection` can include the `opendatacube` function `datacube.Datacube.load()`. It is recommended to use the `dask_chunks` parameter, when loading the data. The function returns a `xarray DataSet`, in order to be compliant with `openeo-processes-dask`, it can be converted to a `DataArray` using the `Dataset.to_array(dim='variable', name=None)` function.
Copy link
Member

Choose a reason for hiding this comment

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

are you using variable in Dataset.to_array(dim='variable', name=None) or bands at EODC?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh, 'variable' is the default in the xarray documentation, we are actually using 'bands'.

update to_array('bands')
@ValentinaHutter
Copy link
Contributor Author

@clausmichele do you want to add a section about client side processing? :)

@clausmichele
Copy link
Member

@ValentinaHutter yes as soon as I have time, it's in my to do list :)

@m-mohr
Copy link
Member

m-mohr commented Oct 2, 2023

Is this still planned to be finished?

@clausmichele
Copy link
Member

Hi @m-mohr, yes it is planned but I totally forgot about it, I will try to update this week.

@m-mohr m-mohr force-pushed the master branch 10 times, most recently from 4ef0b9e to 172284c Compare December 7, 2023 10:51
@clausmichele
Copy link
Member

I finally found the time to do this, once Valentina merges my contribution to her fork, @m-mohr can proceed with the merge of this PR.

Update and rename opendatacube.md to xarray.md
@clausmichele
Copy link
Member

@m-mohr you can merge this now!

@m-mohr
Copy link
Member

m-mohr commented Jun 13, 2024

@clausmichele @ValentinaHutter You need to update the link in .vuepress/config.js. It still points to opendatacube.
Also, to keep old links active, we should add a redirect from the old location to the new location in .vuepress/enhanceApp.js

@m-mohr m-mohr merged commit cc7dd76 into Open-EO:master Jun 13, 2024
@m-mohr
Copy link
Member

m-mohr commented Jun 13, 2024

Thank you everyone!

@ValentinaHutter
Copy link
Contributor Author

I was just about to update the enhanceApp.js, and saw that you did that now, thank you!

@ValentinaHutter ValentinaHutter deleted the patch-1 branch June 13, 2024 09:45
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.

3 participants