Skip to content

Conversation

@forman
Copy link
Member

@forman forman commented Mar 3, 2022

First version of a specification for xcube Multi-Resolution Datasets.

@forman forman self-assigned this Mar 3, 2022
@forman forman removed request for dzelge, gunbra32 and pont-us March 3, 2022 09:11
Copy link
Contributor

@TonioF TonioF left a comment

Choose a reason for hiding this comment

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

Some suggestions.
Also, I take this as a an opportunity to remind of #493

Definition
----------

A xcube _multi-resolution dataset_ refers to an N-D [image
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
A xcube _multi-resolution dataset_ refers to an N-D [image
A xcube _multi-resolution dataset_ uses an N-D [image

Copy link
Member Author

Choose a reason for hiding this comment

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

That is not correct.

forman and others added 2 commits March 3, 2022 10:58
@forman forman merged commit a711fa7 into master Mar 3, 2022
@forman
Copy link
Member Author

forman commented Mar 3, 2022

Some suggestions. Also, I take this as a an opportunity to remind of #493

Thanks. I believe it is better to create a new PR with only applicable changes in.

@forman forman deleted the forman-xxx-mldataset_spec branch March 31, 2022 07:03
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.

4 participants