pysal / tobler

Spatial interpolation, Dasymetric Mapping, & Change of Support
https://pysal.org/tobler
BSD 3-Clause "New" or "Revised" License
145 stars 30 forks source link

fix concatenation issue and copy inputs #80

Closed knaaptime closed 4 years ago

knaaptime commented 4 years ago

this fixes the concatenation index issue, and also it turns out we were also modifying inputs as well

codecov-commenter commented 4 years ago

Codecov Report

Merging #80 into master will increase coverage by 3.60%. The diff coverage is 93.18%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #80      +/-   ##
==========================================
+ Coverage   77.59%   81.19%   +3.60%     
==========================================
  Files          11       11              
  Lines         540      569      +29     
==========================================
+ Hits          419      462      +43     
+ Misses        121      107      -14     
Impacted Files Coverage Δ
tobler/area_weighted/area_weighted.py 87.28% <88.88%> (+6.03%) :arrow_up:
tobler/model/models.py 90.00% <88.88%> (-1.18%) :arrow_down:
tobler/tests/test_interpolators.py 97.95% <100.00%> (+0.13%) :arrow_up:
tobler/util/util.py 85.71% <100.00%> (+12.18%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 11e4d09...f27cc7e. Read the comment docs.

knaaptime commented 4 years ago

resolves #68 #69 and #6