Skip to content

package stars python#135

Merged
edzer merged 5 commits into
edzer:pythonfrom
masawdah:python
Aug 29, 2025
Merged

package stars python#135
edzer merged 5 commits into
edzer:pythonfrom
masawdah:python

Conversation

@masawdah

Copy link
Copy Markdown

No description provided.

@edzer

edzer commented Aug 28, 2025

Copy link
Copy Markdown
Owner

I get on line 866: "NameError: name 'box' is not defined"

@edzer

edzer commented Aug 28, 2025

Copy link
Copy Markdown
Owner

from shapely.geometry import box

@masawdah

Copy link
Copy Markdown
Author

from shapely.geometry import box

I forget this line, also need import numpy as np I've added them

@edzer

edzer commented Aug 28, 2025

Copy link
Copy Markdown
Owner

! NameError: name 'tif' is not defined on line 1002

@masawdah

Copy link
Copy Markdown
Author

! NameError: name 'tif' is not defined on line 1002

just fixed it

@edzer

edzer commented Aug 29, 2025

Copy link
Copy Markdown
Owner

There's still a ! ValueError: cannot convert float NaN to integer coming from

pts = random_points_in_bbox(minx, miny, maxx, maxy, 3)
circles = [p.buffer(500) for p in pts]
gdf = gpd.GeoDataFrame(geometry=circles)

# Aggregate
da.xvec.zonal_stats(
    gdf.geometry,
    stats="max",
    x_coords="x",
    y_coords="y",
)

could you take a look?

@edzer

edzer commented Aug 29, 2025

Copy link
Copy Markdown
Owner

That didn't do it.

@masawdah

Copy link
Copy Markdown
Author

It works on my side maybe it's package/env conflicts

@edzer edzer merged commit 34fcfa6 into edzer:python Aug 29, 2025
@martinfleis

Copy link
Copy Markdown

There's still a ! ValueError: cannot convert float NaN to integer

I believe that @edzer is running an outdated version of Xvec on his machine. This has been reported in xarray-contrib/xvec#89 while it was already fixed on main. I think it is fixed since 0.3.1 (we are on 0.5.1 now) but if that is not the case, I would welcome a bug report on xvec repo.

@edzer

edzer commented Sep 12, 2025

Copy link
Copy Markdown
Owner
$ pip install xvec
Requirement already satisfied: xvec in /home/edzer/.virtualenvs/r-reticulate/lib/python3.9/site-packages (0.3.0)
...

looks like you're right; will dive into this.

@martinfleis

Copy link
Copy Markdown

You need to tell pip to update.

pip install xvec -U

@edzer

edzer commented Sep 12, 2025

Copy link
Copy Markdown
Owner

An old(er) python in the venv was the problem, this runs now; thanks!

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