Skip to content
This repository was archived by the owner on Jan 23, 2025. It is now read-only.

Add first geohash center support#156

Closed
fabienpomerol wants to merge 1 commit intografana:masterfrom
fabienpomerol:feature/first-geohash-center
Closed

Add first geohash center support#156
fabienpomerol wants to merge 1 commit intografana:masterfrom
fabienpomerol:feature/first-geohash-center

Conversation

@fabienpomerol
Copy link
Contributor

We already support the last geohash center, it might be usefull to also center on the first geohash received.

@amotl
Copy link

amotl commented May 11, 2019

Dear Fabien,

we cherry-picked your improvements into our working branch at [1], wrote about this and further updates at [2] and just announced the outcome at [3], which you might also be interested in. Thanks a bunch for sharing your efforts.

With kind regards,
Andreas.

[1] https://github.com/hiveeyes/grafana-worldmap-panel
[2] https://community.grafana.com/t/giving-the-grafana-worldmap-panel-some-love/17210
[3] https://community.hiveeyes.org/t/grafana-worldmap-panel-0-3-0-dev-series/1824

@fabienpomerol fabienpomerol force-pushed the feature/first-geohash-center branch from 904b01d to b32cc68 Compare May 20, 2019 15:22
@fabienpomerol
Copy link
Contributor Author

@amotl i just rebase my code on the master branch, and read again your message saying you already chery-picked this feature 🏎️

Thanks for your effort. Should i close this PR ?

@amotl
Copy link

amotl commented May 21, 2019

Hi @fabienpomerol. As your PR probably received some amendments after being cherry-picked, I believe it is reasonable to close this.

@daniellee promised to review the code which you can pull in the meanwhile from hiveeyes/develop and which should include the feature you added the other day and some more others you might find handy.

Thanks again!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants