[GH-1996]Create object of S2Geography, and implement PoinGeography with its en…
…coder/decoder
Did you read the Contributor Guide?
-
Yes, I have read the Contributor Rules and Contributor Development Guide
-
No, I haven't read it.
Is this PR related to a ticket?
-
Yes, and the PR name follows the format
[SEDONA-XXX] my subject. -
Yes, and the PR name follows the format
[GH-XXX] my subject. -
No:
- this is a documentation update. The PR name follows the format
[DOCS] my subject - this is a CI update. The PR name follows the format
[CI] my subject
- this is a documentation update. The PR name follows the format
What changes were proposed in this PR?
How was this patch tested?
Did this PR include necessary documentation updates?
- Yes, I am adding a new API. I am using the current SNAPSHOT version number in
vX.Y.Zformat. - Yes, I have updated the documentation.
- No, this PR does not affect any public API so no need to change the documentation.
@ZhuochengShang great work. Please create a GitHub issue and link the PR to that issue using [GH-XXXX]
Great work @ZhuochengShang . It is great to have a working PR that shows the progress. But when it is ready, we'd like to break it into a sequence of small PRs. This will make the review process easier.
Please also update the title of this PR to [GH-XXXX] that links to the GitHub issue.
Thank you for this work so far! Just a note that if you (or Kristin or Jia) run across anything that doesn't make sense or is sub-optimial in the C++ version of S2Geography to let me know!
What is the status of this PR?
Awesome!