Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add geo::HasKernel interop #122

Draft
wants to merge 8 commits into
base: master
Choose a base branch
from

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.

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