Michael Kirk

> I'm not sure I fully agree about the zoom expected behavior (especially when you consider mobile experience as well, where zoom can also rotate the map). Mobile is almost...

> I don't have apple maps, but if you zoom enough in google maps the lock is disabled at some point. Same goes for waze. So no, this argument doesn't...

While looking into this, I've noticed some of the typing is wrong or incomplete. Are you open to PR's that simply add more complete and accurate typings without functional changes?

I wanted to note (and for searchability), the tests currently fail to build for me due to some deprecated code in robin-hood-hashing: ``` [188/502] Building CXX object test/CMakeFiles/idtable.dir/idtable.cc.o FAILED: test/CMakeFiles/idtable.dir/idtable.cc.o...

> you can avoid it on your platform by turning off -Werror which you can do when you run cmake by passing different options we have set up. I'd assumed...

With [this little addendum](https://github.com/valhalla/valhalla/pull/4698#discussion_r1575399301), #4698 does the trick! Thanks both. I still don't understand why the the compiler warnings are causing the build to fail when I haven't explicitly set...

Some notes from the geo transition to merge queues (which seem to be working fine BTW). They are my own notes and might not be 100% correct: Within repo >...

That error message doesn't appear in this crate, but it does appear in geo's implementation of booleanops. It's possible you are experiencing https://github.com/georust/geo/issues/1103

In theory I'm on board. Can you also include instructions and/or a script for the maintainers to run when we want to refresh the bindings? > I think we want...

> Should we surface this buildtime_bindgen feature in the proj crate as well? My understanding is that most people who use proj in rust are using the `proj` crate, not...