Skip to content

Fix: Update h3 imports to work with h3 v4.1.0+#58

Merged
Bondify merged 1 commit intoBondify:masterfrom
teleksterling:patch-1
Jan 27, 2025
Merged

Fix: Update h3 imports to work with h3 v4.1.0+#58
Bondify merged 1 commit intoBondify:masterfrom
teleksterling:patch-1

Conversation

@teleksterling
Copy link
Copy Markdown
Contributor

Replaced references to the two functions imported from h3:

  • geo_to_h3 -> latlng_to_cell
  • k_ring -> grid_ring

Addresses #54.

See also original comment by @dfellis in uber/h3-py#100 (comment)

Replaced references to the two imported functions:
* geo_to_h3 -> latlng_to_cell
* k_ring -> grid_ring
Addresses #100
@fedderw
Copy link
Copy Markdown

fedderw commented Jan 24, 2025

I hate to bother anyone for not keeping a fun side-project perfectly-up-to-date with the latest dependency changes, as I have a graveyard of such things.

In fact, I'm getting back to one now that uses this library, so I hope @Bondify @Rikuoja don't mind if I bump this PR for them. 😄

It's a super small, 3-line change to the names of some function calls, if that helps!

@Bondify
Copy link
Copy Markdown
Owner

Bondify commented Jan 27, 2025

This update is due. Thanks @teleksterling for the PR and @fedderw for the reminder. I'm not giving this project as much love as I'd like lately 🙏

@Bondify Bondify merged commit e2b2bda into Bondify:master Jan 27, 2025
@fedderw
Copy link
Copy Markdown

fedderw commented Feb 25, 2025

This update is due. Thanks @teleksterling for the PR and @fedderw for the reminder. I'm not giving this project as much love as I'd like lately 🙏

I just came back today and saw it got merged - thank you! (And totally understand)

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.

3 participants