Skip to content

Add geo::HasKernel interop#122

Closed
MultisampledNight wants to merge 8 commits intoreem:masterfrom
MultisampledNight:geo-interop
Closed

Add geo::HasKernel interop#122
MultisampledNight wants to merge 8 commits intoreem:masterfrom
MultisampledNight:geo-interop

Conversation

@MultisampledNight
Copy link
Contributor

Closes #121. Somehow NotNan doesn't impl Float and as such it doesn't like my HasKernel impl, so I left that out for now until I have time to actually properly look at this.

@MultisampledNight
Copy link
Contributor Author

Tests fail due to a rename of clippy lints, not my changes. Should include a rename either way?

@mbrubeck
Copy link
Collaborator

mbrubeck commented Mar 9, 2023

Yes, feel free to update that lint in this PR (or as a separate PR if you prefer).

@MultisampledNight
Copy link
Contributor Author

MultisampledNight commented Mar 9, 2023

Technically this is ready and okay so far, but according to the geo maintainers on Discord they want to change their RobustKernel to remove the Float requirement, so this way the vendored NotNanRobustKernel wouldn't be required.

@MultisampledNight MultisampledNight closed this by deleting the head repository Jan 30, 2026
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.

OrderedFloat and NotNan should implement geo::HasKernel (behind feature)

2 participants