Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@martinfleis
Copy link
Member

Index-oriented and vectorized alternative of get_node_id. Also fixes the user guide failure.

@martinfleis martinfleis added this to the 0.8.0 milestone Jun 14, 2024
@martinfleis martinfleis requested a review from jGaboardi June 14, 2024 18:16
@martinfleis martinfleis self-assigned this Jun 14, 2024
- name: setup micromamba
uses: mamba-org/setup-micromamba@v1
with:
environment-file: ci/envs/312-latest.yaml
Copy link
Member Author

Choose a reason for hiding this comment

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

I am using some of the geopandas 1.0 api (union_all). GeoPandas will be released sooner than momepy.

@codecov
Copy link

codecov bot commented Jun 14, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.0%. Comparing base (4037c70) to head (bdc8d6e).
Report is 85 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##            main    #600     +/-   ##
=======================================
+ Coverage   97.4%   98.0%   +0.6%     
=======================================
  Files         26      37     +11     
  Lines       4328    6067   +1739     
=======================================
+ Hits        4214    5945   +1731     
- Misses       114     122      +8     
Files with missing lines Coverage Δ
momepy/functional/_elements.py 99.1% <100.0%> (ø)
momepy/functional/tests/test_elements.py 98.9% <100.0%> (ø)

@martinfleis martinfleis merged commit f2eb9b4 into pysal:main Jun 14, 2024
@martinfleis martinfleis deleted the get_nearest_node branch June 14, 2024 20:50
@martinfleis martinfleis added the enhancement New feature or request label Jun 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants