oturns / geosnap

The Geospatial Neighborhood Analysis Package
https://oturns.github.io/geosnap-guide
BSD 3-Clause "New" or "Revised" License
247 stars 32 forks source link

refactor #158

Closed knaaptime closed 4 years ago

knaaptime commented 5 years ago

update - changes in this pr:

refactor

test suite

docs

misc

knaaptime commented 5 years ago

because i dont want them to show up in tab completion

knaaptime commented 5 years ago

(otherwise e.g. you'd get both geosnap.community and geosnap.Community when you tab)

knaaptime commented 5 years ago

i want to trigger a travis build, so i'll do the docs in a separate PR

knaaptime commented 5 years ago

tests pass locally but i cant get the build to trigger on travis...

weikang9009 commented 5 years ago

The travis CI seems to have not been triggered for the past three months. The last CI build on the master branch was from PR https://github.com/spatialucr/geosnap/pull/131

knaaptime commented 5 years ago

good catch. Looks like @suhanmappingideas got one to trigger from the vizsu branch and that seems to have gotten things moving. how strange

knaaptime commented 5 years ago

I reworked the tests and CI config so that as long as you have the LTDB and NCDB environment variables set, tests should pass on forks

(everything is passing on mine) this should be ready to go