Skip to content

Issue872#878

Merged
gadieid merged 3 commits into
masterfrom
issue872
May 24, 2026
Merged

Issue872#878
gadieid merged 3 commits into
masterfrom
issue872

Conversation

@ilayfalach
Copy link
Copy Markdown
Collaborator

No description provided.

Ilay Falach and others added 3 commits May 17, 2026 15:01
- topography.py: initialize elevation column as float64 (0.0 instead of
  0) to prevent TypeError/FutureWarning when writing interpolated floats
  into an int64 column (Pandas 2+/3 compatibility)

- analysis.py: replace deprecated pd.api.types.is_datetime64tz_dtype()
  with isinstance(dtype, pd.DatetimeTZDtype) (Pandas4Warning)

- presentationLayer.py: replace matplotlib.cm.get_cmap with
  matplotlib.colormaps; fix dask.dataframe.core.DataFrame reference to
  dask.dataframe.DataFrame; fix plotScatter to guard against empty
  DataFrames and use .to_numpy() to avoid duplicate-index reindex errors
  in Pandas 3

- demography.py: fix plotPopulationInPolygon to set CRS without
  transforming when inputCRS is None, preventing "Cannot transform naive
  geometries" ValueError

- test_demography.py: add getImageFromCorners to MockTilesToolkit;
  correct small_polygon_itm fixture to intersect only one data block so
  partial-coverage assertion holds

- pytest.ini: register 'unit' marker to silence PytestUnknownMarkWarning

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Project geometries to ITM (EPSG:2039) before computing .area so that
areaFraction and population estimates are based on metric units instead
of degrees, eliminating the UserWarning and incorrect values.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@gadieid gadieid merged commit 7a57a11 into master May 24, 2026
@gadieid gadieid deleted the issue872 branch May 24, 2026 13:08
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.

2 participants