pysal / momepy

Urban Morphology Measuring Toolkit
https://docs.momepy.org
BSD 3-Clause "New" or "Revised" License
478 stars 58 forks source link

MAINT: compatibility, warning cleanup, test lint #574

Closed martinfleis closed 5 months ago

martinfleis commented 5 months ago

I wanted to primarily cleanup warnings from our CI. So some changes related to that, some changes related to compatibility (FutureWarning removal), plus linting of tests which was for some reason disabled.

codecov[bot] commented 5 months ago

Codecov Report

Attention: Patch coverage is 98.18182% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 97.8%. Comparing base (4037c70) to head (9d1d944). Report is 18 commits behind head on main.

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/pysal/momepy/pull/574/graphs/tree.svg?width=650&height=150&src=pr&token=VNn0WR5JWT&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pysal)](https://app.codecov.io/gh/pysal/momepy/pull/574?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pysal) ```diff @@ Coverage Diff @@ ## main #574 +/- ## ======================================= + Coverage 97.4% 97.8% +0.4% ======================================= Files 26 37 +11 Lines 4328 5160 +832 ======================================= + Hits 4214 5045 +831 - Misses 114 115 +1 ``` | [Files](https://app.codecov.io/gh/pysal/momepy/pull/574?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pysal) | Coverage Δ | | |---|---|---| | [momepy/coins.py](https://app.codecov.io/gh/pysal/momepy/pull/574?src=pr&el=tree&filepath=momepy%2Fcoins.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pysal#diff-bW9tZXB5L2NvaW5zLnB5) | `95.3% <ø> (ø)` | | | [momepy/dimension.py](https://app.codecov.io/gh/pysal/momepy/pull/574?src=pr&el=tree&filepath=momepy%2Fdimension.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pysal#diff-bW9tZXB5L2RpbWVuc2lvbi5weQ==) | `98.8% <100.0%> (+<0.1%)` | :arrow_up: | | [momepy/functional/\_elements.py](https://app.codecov.io/gh/pysal/momepy/pull/574?src=pr&el=tree&filepath=momepy%2Ffunctional%2F_elements.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pysal#diff-bW9tZXB5L2Z1bmN0aW9uYWwvX2VsZW1lbnRzLnB5) | `100.0% <100.0%> (ø)` | | | [momepy/functional/tests/test\_elements.py](https://app.codecov.io/gh/pysal/momepy/pull/574?src=pr&el=tree&filepath=momepy%2Ffunctional%2Ftests%2Ftest_elements.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pysal#diff-bW9tZXB5L2Z1bmN0aW9uYWwvdGVzdHMvdGVzdF9lbGVtZW50cy5weQ==) | `100.0% <ø> (ø)` | | | [momepy/intensity.py](https://app.codecov.io/gh/pysal/momepy/pull/574?src=pr&el=tree&filepath=momepy%2Fintensity.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pysal#diff-bW9tZXB5L2ludGVuc2l0eS5weQ==) | `99.5% <100.0%> (+<0.1%)` | :arrow_up: | | [momepy/preprocessing.py](https://app.codecov.io/gh/pysal/momepy/pull/574?src=pr&el=tree&filepath=momepy%2Fpreprocessing.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pysal#diff-bW9tZXB5L3ByZXByb2Nlc3NpbmcucHk=) | `93.4% <100.0%> (+0.1%)` | :arrow_up: | | [momepy/tests/test\_dimension.py](https://app.codecov.io/gh/pysal/momepy/pull/574?src=pr&el=tree&filepath=momepy%2Ftests%2Ftest_dimension.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pysal#diff-bW9tZXB5L3Rlc3RzL3Rlc3RfZGltZW5zaW9uLnB5) | `100.0% <ø> (ø)` | | | [momepy/tests/test\_distribution.py](https://app.codecov.io/gh/pysal/momepy/pull/574?src=pr&el=tree&filepath=momepy%2Ftests%2Ftest_distribution.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pysal#diff-bW9tZXB5L3Rlc3RzL3Rlc3RfZGlzdHJpYnV0aW9uLnB5) | `100.0% <100.0%> (ø)` | | | [momepy/tests/test\_diversity.py](https://app.codecov.io/gh/pysal/momepy/pull/574?src=pr&el=tree&filepath=momepy%2Ftests%2Ftest_diversity.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pysal#diff-bW9tZXB5L3Rlc3RzL3Rlc3RfZGl2ZXJzaXR5LnB5) | `100.0% <100.0%> (ø)` | | | [momepy/tests/test\_elements.py](https://app.codecov.io/gh/pysal/momepy/pull/574?src=pr&el=tree&filepath=momepy%2Ftests%2Ftest_elements.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pysal#diff-bW9tZXB5L3Rlc3RzL3Rlc3RfZWxlbWVudHMucHk=) | `100.0% <100.0%> (ø)` | | | ... and [9 more](https://app.codecov.io/gh/pysal/momepy/pull/574?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pysal) | |