fixes #680

[speco29]

Added the missing dependency

[martinfleis]

You don't need to import it. You just need to ensure it is installed.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.1%. Comparing base (4037c70) to head (3c05758).
Report is 102 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##            main    #681     +/-   ##
=======================================
+ Coverage   97.4%   98.1%   +0.8%     
=======================================
  Files         26      40     +14     
  Lines       4328    7208   +2880     
=======================================
+ Hits        4214    7074   +2860     
- Misses       114     134     +20     

see 1 file with indirect coverage changes

[speco29]

You don't need to import it. You just need to ensure it is installed.

Should I close this issue then?
but I think having known which library is being used in the examples would be much better.
let me know your thoughts;)

[martinfleis]

If you want to do something about #680, then add a note somewhere that mapclassify needs to be installed to run the notebooks in the user guide. Do not import it, the linter fails on that.

However, if you install momepy the recommended way from conda-forge, it will pull mapclassify automatically, so I am not sure if this is even worth doing.