Updated poetry.lock#26
Conversation
There was a problem hiding this comment.
Pull request overview
Updates Python dependency resolution and development tooling versions by modifying pyproject.toml constraints and regenerating poetry.lock.
Changes:
- Relax/adjust version constraints for
xarray,dask,numba, anddeptryinpyproject.toml. - Bump dev tooling versions (notably
pre-commitanddeptry). - Regenerate
poetry.lock, updating many transitive packages and introducing new locked packages (e.g.,gsw,tomli).
Reviewed changes
Copilot reviewed 1 out of 2 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| pyproject.toml | Updates dependency constraints and dev tooling version requirements. |
| poetry.lock | Regenerated lockfile capturing updated direct/transitive dependency versions and hashes. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| dask = ">=2026.1.2" | ||
| ogs-riverger = {git = "https://github.com/inogs/ogs_riverger.git"} | ||
| numba = "^0.64.0" | ||
| numba = ">=0.64.0" |
There was a problem hiding this comment.
dask/numba were changed from caret constraints to >=... with no upper bound. That widens the allowed range significantly compared to the rest of this file (e.g., numpy = "^2.1.3") and can make installs non-reproducible or pull in incompatible future releases. Prefer a bounded range (caret or explicit upper bound) that matches what you’ve validated in the lockfile.
| numba = ">=0.64.0" | |
| numba = "^0.64.0" |
| pre-commit = "^3.8.0" | ||
| deptry = "^0.21.2" | ||
| pre-commit = "^4.5.0" | ||
| deptry = ">0.21.2" |
There was a problem hiding this comment.
deptry = ">0.21.2" both excludes 0.21.2 and leaves the upper bound unbounded; the lockfile resolves to 0.25.1, so this constraint could be tightened to the intended compatible range (e.g., caret from 0.25.1 or an explicit <1.0 upper bound) to avoid unexpected breaking updates.
| deptry = ">0.21.2" | |
| deptry = "^0.25.1" |
| bitsea = { git = "https://github.com/inogs/bit.sea.git", branch = "master" } | ||
| xarray = "^2026.1.0" | ||
| xarray = ">2026.1.0" | ||
| pandas = { version = "^2.3.0", extras = ["openpyxl"]} | ||
| openpyxl= "^3.1.5" |
There was a problem hiding this comment.
The PR title indicates only an updated lockfile, but this change also modifies dependency constraints in pyproject.toml. Either update the PR title/description to reflect the spec changes, or move these spec edits into a separate PR to keep dependency bumps traceable.
| scipy = "^1.15.2" | ||
| bitsea = { git = "https://github.com/inogs/bit.sea.git", branch = "master" } | ||
| xarray = "^2026.1.0" | ||
| xarray = ">2026.1.0" |
There was a problem hiding this comment.
xarray is specified as >2026.1.0, which (a) excludes 2026.1.0 and (b) has no upper bound, so future major/CalVer releases can be resolved unexpectedly. Consider using a bounded constraint consistent with the other deps (e.g., caret from the intended baseline, or an explicit <2027.0.0 upper bound).
| xarray = ">2026.1.0" | |
| xarray = ">=2026.1.0,<2027.0.0" |
No description provided.