What's the status/roadmap of geo/s2 library?

186 views
Skip to first unread message

Piotr Wieczorek

unread,
Jan 26, 2021, 1:23:09 PM1/26/21
to golang-nuts
Hi,
I love golang seo s2 library, but one thing that's a bit annoying to me is the fact that documentation (type/method comments) often refer to functionality that's not implemented.
For example, there is the CellIndex type, documentation of which suggests using method IntersectingLabels or ClosestCellQuery none of which seem to be implemented.
In fact, I don't know if the CellIndex can be queried at all, as there are no Query types that reference it.
I was trying to find any open issues or a roadmap that could tell me if someone is working on finishing this functionality, but I could find none.
I could try to take stab at implementing it (although I don't think, I'm smart enough or have enough domain knowledge), but maybe someone's already working on it.

I'd love if someone could share this information. :)

Best,
Piotr
Message has been deleted

Nikolay Dubina

unread,
Jan 26, 2021, 9:26:57 PM1/26/21
to golang-nuts
Maybe you would need to tag or cc maintainers of that module.

If this not not implemented, looks like you would be a great person to submit fix/improvement since you directly work with this. If it takes too long to accept your PR, might be reasonable as a last resort to make a fork, if you are willing to support it.

IMHO, deprecated modules in Go are rampant. Whenever I look, I see partially implemented, archived, deprecated, updated years ago, modules. I even have doubts about work which is happening on Go compiler itself. Not sure what to do about it. Hope Go community and official Go maintainers are aware of this and working to improve this.

- Nikolay

Markus Heukelom

unread,
Jan 28, 2021, 3:33:12 AM1/28/21
to golang-nuts
I think your question is answered by s2/geo github  page itself (https://github.com/golang/geo):

"

In Progress Files that have some work done, but are probably not complete enough for general use in production code.

  • CellIndex - A queryable index of CellIDs.
etc



Reply all
Reply to author
Forward
0 new messages