Skip to content

Conversation

@mileswwatkins
Copy link
Contributor

I noticed that it was on the to-do list, and thought it'd be a nice feature to include.

I took a guess as to how you might like to handle imprecise geocoding, but let me know if you'd like me to tweak that Exception or any other piece.

@mileswwatkins
Copy link
Contributor Author

Hmm, looks like Travis hit a bunch of unrelated bugs in tests.test_image.TestProcess that didn't pop up for me locally.

@scisco
Copy link
Contributor

scisco commented Oct 15, 2015

Will reopen if the PR is updated to pass tests

@scisco scisco closed this Oct 15, 2015
@mileswwatkins mileswwatkins reopened this Oct 27, 2015
@mileswwatkins
Copy link
Contributor Author

The tests pass locally for me; I think the push CI test isn't passing because my PR's coming from an external (forked) repo instead of the main developmentseed repo.

@scisco
Copy link
Contributor

scisco commented Oct 29, 2015

Moving this to #125

@scisco scisco closed this Oct 29, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants