Skip to content

add support for python 3.14#117

Merged
ValentinaHutter merged 4 commits into
Open-EO:mainfrom
vincentsarago:feature/add-python314-support
Mar 30, 2026
Merged

add support for python 3.14#117
ValentinaHutter merged 4 commits into
Open-EO:mainfrom
vincentsarago:feature/add-python314-support

Conversation

@vincentsarago

Copy link
Copy Markdown
Contributor

This PR add python 3.14 support and open Dask dependency

Note: the xarray and Dask have a pretty low usage footprint, only used 👇

if isinstance(result, xr.DataArray):
processed_result = {
"entity_type": "xarray.DataArray",
"info": {
"shape": result.shape,
"dimensions": list(result.dims),
# "attributes": result.attrs,
"dtype": str(result.dtype),
},
}
elif isinstance(result, da.Array):
processed_result = {
"entity_type": "dask.Array",
"info": {
"shape": result.shape,
"dtype": str(result.dtype),
"chunk_size": result.chunksize,
"chunk_type": type(result._meta).__name__,
},
}

the risk of breaking change in both xarray.DataArray and dask.array is pretty low with the usage made in this library

Comment thread pyproject.toml
python = ">=3.10,<3.15"
pydantic = "^2.4.0"
pyproj = "^3.4.0"
pyproj = ">=3.4.0,<4.0"

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

same, pyproj is only used for CRS parsing. setting upper version to 4.0 is safe

@vincentsarago vincentsarago force-pushed the feature/add-python314-support branch from c4936ac to 353768b Compare March 26, 2026 14:52
@vincentsarago vincentsarago force-pushed the feature/add-python314-support branch from dcd2319 to ff09286 Compare March 26, 2026 15:47
@vincentsarago

Copy link
Copy Markdown
Contributor Author

@ValentinaHutter sorry for yet another PR 🙈

@ValentinaHutter ValentinaHutter merged commit f3a9409 into Open-EO:main Mar 30, 2026
5 checks passed
Comment thread pyproject.toml
pyproj = "^3.4.0"
pyproj = ">=3.4.0,<4.0"
networkx = ">=3.0.0"
geojson-pydantic = "^1.0.0"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@vincentsarago sorry for the late response to this PR, but you had some changes in geojson-pydantic 2.1 concerning Python 3.14. Should this dependency be bumped to >= 2.1, or is 1.x sufficient?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

we don't HAVE TO but I think it wouldn't arm to allow both 1.x and 2.x

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