-
Notifications
You must be signed in to change notification settings - Fork 1
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
Implement open_data and support tiff, netcdf and zarr for s3 and https protocol; Implement stack-mode which stacks items containing tiffs; #19
Merged
Merged
Changes from 21 commits
Commits
Show all changes
24 commits
Select commit
Hold shift + click to select a range
d8f2685
tiff works, netcdf not
konstntokas 51293c4
opener for https added
konstntokas 1bbb317
ready for rebase
konstntokas 14fc51b
backup
konstntokas 5002dcf
new workflow
konstntokas 35c14f1
new workflow 2
konstntokas 9609b95
need for checkout
konstntokas 942f8c8
open_params available for all data types and formats
konstntokas 8ece8d0
workflow fix0
konstntokas 019f31a
run xcube server in CI
konstntokas d78f06f
run server in background
konstntokas 21f1a88
all tests are fine
konstntokas dd2e412
ready for review
konstntokas b11304f
added cassette for unittest
konstntokas 16ec6f4
stacking mode implemented
konstntokas 82fd9e5
odc added to the requirement
konstntokas 1eaecd0
unittesting done for stack-mode
konstntokas 0a09f8e
all notebooks are running
konstntokas 6740bd1
README updated
konstntokas e19624b
meeting with Norman and Pont recap part 1
konstntokas 3c57f02
mldataset precalculated with finest resolution found in the bands
konstntokas 5486a5b
Norman's review addressed
konstntokas fb467c4
badges added
konstntokas 3ba749b
cov badge link updated
konstntokas File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,13 @@ | ||
# Code of Conduct | ||
|
||
We are committed to providing a friendly, safe, and welcoming environment | ||
for all, regardless of gender, sexual orientation, ability, ethnicity, | ||
religion, or any other characteristic. | ||
|
||
We expect everyone to treat each other with respect and kindness. We do not | ||
tolerate harassment or discrimination of any kind. | ||
|
||
If you witness or experience any behavior that violates this code of conduct, | ||
please report it to the project maintainers immediately. | ||
|
||
Thank you for helping us create a welcoming community! |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,42 @@ | ||
# How to contribute | ||
|
||
The `xcube-stac` project welcomes contributions of any form | ||
as long as you respect our [code of conduct](CODE_OF_CONDUCT.md) and stay | ||
in line with the following instructions and guidelines. | ||
|
||
If you have suggestions, ideas, feature requests, or if you have identified | ||
a malfunction or error, then please | ||
[post an issue](https://github.com/xcube-dev/xcube-stac/issues). | ||
|
||
If you'd like to submit code or documentation changes, we ask you to provide a | ||
pull request (PR) | ||
[here](https://github.com/xcube-dev/xcube-stac/pulls). | ||
For code and configuration changes, your PR must be linked to a | ||
corresponding issue. | ||
|
||
To ensure that your code contributions are consistent with our project’s | ||
coding guidelines, please make sure all applicable items of the following | ||
checklist are addressed in your PR. | ||
|
||
**PR checklist** | ||
|
||
* Format code using [black](https://black.readthedocs.io/) with default settings. | ||
Check also section [code style](#code-style) below. | ||
* Your change shall not break existing unit tests. | ||
`pytest` must run without errors. | ||
* Add unit tests for any new code not yet covered by tests. | ||
* Make sure test coverage is close to 100% for any change. | ||
Use `pytest --cov=xcube_stac --cov-report=html` to verify. | ||
|
||
## Code style <a name="code-style"></a> | ||
|
||
The `xcube-stac` code compliant to [PEP-8](https://pep8.org/) except for a line | ||
length of 88 characters as recommended by [black](https://black.readthedocs.io/). | ||
Since black is un-opinionated regarding the order of imports, | ||
we use the following three import blocks separated by an empty | ||
line: | ||
|
||
1. Python standard library imports, e.g., `os`, `typing`, etc | ||
2. 3rd-party imports, e.g., `xarray`, `zarr`, etc | ||
3. Relative `xcube_stac` module imports using `.` prefix. | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,13 @@ | ||
# Security Policy | ||
|
||
## Supported Versions | ||
|
||
| Version | Supported | | ||
|---------| ------------------ | | ||
| 0.1.x | :white_check_mark: | | ||
|
||
## Reporting a Vulnerability | ||
|
||
To report a vulnerability, please post and [issue](https://github.com/xcube-dev/xcube-stac/issues) | ||
and use prefix `[SECURITY]` for the title. Security issues will be treated | ||
with high priority. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change | ||||
---|---|---|---|---|---|---|
@@ -0,0 +1,56 @@ | ||||||
name: xcube-stac | ||||||
channels: | ||||||
- conda-forge | ||||||
- defaults | ||||||
dependencies: | ||||||
# Python | ||||||
- python >=3.9 | ||||||
# Required xcube | ||||||
- affine >=2.2 | ||||||
- botocore >=1.34.51 | ||||||
- cftime >=1.6.3 | ||||||
- click >=8.0 | ||||||
- cmocean >=2.0 | ||||||
- dask >=2021.6 | ||||||
- dask-image >=0.6 | ||||||
- deprecated >=1.2 | ||||||
- distributed >=2021.6 | ||||||
- fiona >=1.8 | ||||||
- fsspec >=2021.6 | ||||||
- gdal >=3.0 | ||||||
- geopandas >=0.8 | ||||||
- jdcal >=1.4 | ||||||
- jsonschema >=3.2 | ||||||
- mashumaro | ||||||
- matplotlib-base >=3.8.3 | ||||||
- netcdf4 >=1.5 | ||||||
- numba >=0.52 | ||||||
- numcodecs >=0.12.1 | ||||||
- numpy >=1.16 | ||||||
- pandas >=1.3 | ||||||
- pillow >=6.0 | ||||||
- pyjwt >=1.7 | ||||||
- pyproj >=3.0 | ||||||
- pyyaml >=5.4 | ||||||
- rasterio >=1.2 | ||||||
- requests >=2.25 | ||||||
- rfc3339-validator >=0.1 # for python-jsonschema date-time format validation | ||||||
- rioxarray >=0.11 | ||||||
- s3fs >=2021.6 | ||||||
- setuptools >=41.0 | ||||||
- shapely >=1.6 | ||||||
- tornado >=6.0 | ||||||
- urllib3 >=1.26 | ||||||
- xarray >=2022.6 | ||||||
- zarr >=2.11 | ||||||
# Required xcube-stac | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Done |
||||||
- odc-geo | ||||||
- odc-stac | ||||||
- pystac | ||||||
- pystac-client | ||||||
# for testing | ||||||
- black | ||||||
- flake8 | ||||||
- pytest | ||||||
- pytest-cov | ||||||
- pytest-recording |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done