-
Notifications
You must be signed in to change notification settings - Fork 10
Support icechunk #101
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
base: main
Are you sure you want to change the base?
Support icechunk #101
Changes from all commits
441063e
5d760bc
b87582d
7890228
ad28eab
9161be4
a018066
3e57c11
4d20b59
21f33d1
e6d86ea
d99859c
791d618
e1d90bf
98809ae
89b1755
ab81349
de5e640
900f2d5
accce17
9d7c6a8
cbd9105
32a9c43
9aed634
32f3788
3c255b6
d246e9a
5ebb636
0cfe71d
e3da262
08e2aed
c9b731b
b966716
5d95982
59b7e29
973d53f
26af58b
06e1d20
a4852af
d1bf556
fe5e3e1
e1756d0
e9a1d48
f295ded
b74fa6a
f2c935a
ee21baa
0141b51
ae5480c
4d2735b
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -105,5 +105,7 @@ cdk.out/ | |
| node_modules | ||
| cdk.context.json | ||
| *.nc | ||
| .DS_Store | ||
|
|
||
| .test-deploy-env | ||
| .vscode/settings.json | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -25,8 +25,8 @@ classifiers = [ | |
| ] | ||
| dynamic = ["version"] | ||
| dependencies = [ | ||
| "titiler.core>=0.23.0,<0.24", | ||
| "titiler.xarray>=0.23.0,<0.24", | ||
| "titiler-core>=0.25.0,<0.26", | ||
| "titiler-xarray>=0.25.0,<0.26", | ||
| "aiohttp", | ||
| "aiobotocore>=2.24.0", | ||
| "boto3>=1.39.0", | ||
|
|
@@ -41,8 +41,9 @@ dependencies = [ | |
| "requests", | ||
| "rioxarray", | ||
| "s3fs", | ||
| "xarray", | ||
| "zarr>=2,<3", | ||
| "xarray>=2025.10.1", | ||
| "zarr>3.1.0", | ||
| "icechunk>=1.1.9", | ||
| ] | ||
|
|
||
| [project.optional-dependencies] | ||
|
|
@@ -56,6 +57,7 @@ lambda = [ | |
|
|
||
| [dependency-groups] | ||
| dev = [ | ||
| "dask>=2025.9.1", | ||
|
Collaborator
Author
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. I think this was beneficial for generating the test fixtures, but happy to remove it if you think that I should.
Contributor
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. I think it is fine to include it in the dev dependencies |
||
| "fakeredis>=2.23.5", | ||
| "httpx", | ||
| "ipykernel>=6.30.1", | ||
|
|
@@ -65,15 +67,23 @@ dev = [ | |
| "pytest-asyncio>=0.24.0", | ||
| "pytest-cov>=5.0.0", | ||
| "pytest>=8.3.2", | ||
| "pytest-xdist", | ||
| "uvicorn>=0.34.0", | ||
| "yappi>=1.6.0", | ||
| "virtualizarr", | ||
| "obstore", | ||
| ] | ||
| deployment = [ | ||
| "aws-cdk-lib~=2.177.0", | ||
| "constructs>=10.4.2", | ||
| "pydantic-settings~=2.0", | ||
| "python-dotenv>=1.0.1", | ||
| ] | ||
| notebooks = [ | ||
| "folium", | ||
| "httpx", | ||
| "matplotlib", | ||
| ] | ||
|
|
||
| [project.urls] | ||
| Homepage = "https://github.com/developmentseed/titiler-multidim" | ||
|
|
@@ -97,6 +107,8 @@ known_first_party = ["titiler"] | |
| default_section = "THIRDPARTY" | ||
|
|
||
| [tool.ruff] | ||
|
|
||
| [tool.ruff.lint] | ||
| select = [ | ||
| "D1", # pydocstyle errors | ||
| "E", # pycodestyle errors | ||
|
|
@@ -111,7 +123,7 @@ ignore = [ | |
| "B905", # ignore zip() without an explicit strict= parameter, only support with python >3.10 | ||
| ] | ||
|
|
||
| [tool.ruff.per-file-ignores] | ||
| [tool.ruff.lint.per-file-ignores] | ||
| "tests/test_app.py" = ["D1"] | ||
| "tests/fixtures/generate_test_zarr.py" = ["D1"] | ||
| "tests/fixtures/generate_test_reference.py" = ["D1"] | ||
|
|
@@ -126,8 +138,6 @@ explicit_package_bases = true | |
| requires = ["pdm-backend"] | ||
| build-backend = "pdm.backend" | ||
|
|
||
|
|
||
|
|
||
| [tool.pdm.version] | ||
| source = "file" | ||
| path = "src/titiler/multidim/__init__.py" | ||
|
|
@@ -142,3 +152,4 @@ excludes = ["tests/", "**/.mypy_cache", "**/.DS_Store"] | |
|
|
||
| [tool.pytest.ini_options] | ||
| addopts = "--cov=titiler.multidim --cov-report term-missing -s -vv" | ||
| pythonpath = ["src", "tests"] | ||
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.
Added pytest-xdist mostly for more performant tests locally. Does not do much on a 2 core gh runner.