Skip to content

Shape notebook - #439

Merged
ljwolf merged 5 commits into
pysal:mainfrom
leehach:shape-notebook
Mar 20, 2026
Merged

Shape notebook#439
ljwolf merged 5 commits into
pysal:mainfrom
leehach:shape-notebook

Conversation

@leehach

@leehach leehach commented Mar 18, 2026

Copy link
Copy Markdown
Contributor

Changed shape_measures.ipynb to use Virginia counties instead of Mississippi counties, primarily because Virginia is a builtin dataset, whereas Mississippi required downloading south.shp and filtering. Also, Virginia has counties with holes, demonstrating that shape measures can be calculated for polygons with holes.

Some edits to existing function demos. Also reprojected from WGS84 to World Mercator, and discuss recommendation to use a coformal projection for shape measures.

Added several examples of inertial measures, demonstrating capabilities added as part of PR #420.

@codecov

codecov Bot commented Mar 18, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.2%. Comparing base (2399b4e) to head (39b59d2).
⚠️ Report is 17 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@          Coverage Diff          @@
##            main    #439   +/-   ##
=====================================
  Coverage   82.2%   82.2%           
=====================================
  Files         27      27           
  Lines       3969    3969           
=====================================
  Hits        3264    3264           
  Misses       705     705           

see 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ljwolf
ljwolf merged commit e0975f5 into pysal:main Mar 20, 2026
17 checks passed
@leehach
leehach deleted the shape-notebook branch March 21, 2026 16:15
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