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.

Sign up for GitHub

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

Jump to bottom

Remove some polymorphic comparison functions #15

Merged

patricoferris merged 1 commit into main from no-poly-funcs

Aug 10, 2023

Contributor

patricoferris commented Aug 10, 2023

Seems to shave off some time in the microbenchmarks for find.


          Remove some polymorphic comparison functions

69e4c4b

patricoferris merged commit c85fe48 into main

1 check passed

patricoferris deleted the no-poly-funcs branch

August 10, 2023 18:35

patricoferris added a commit to patricoferris/opam-repository that referenced this pull request


          [new release] rtree (0.1.1)

d828602

CHANGES:

## Bugs, Fixes and Optimisations

 - Remove extra length calculations (geocaml/ocaml-rtree#16, @lindig)
 - Remove some polymorphic comparisons and replace with `Float` functions (geocaml/ocaml-rtree#15, @patricoferris)
 - Fix stack overflows in OMT and Rectangle.merge (geocaml/ocaml-rtree#14, @patricoferris, reported by @lindig)

patricoferris mentioned this pull request

[new release] rtree (0.1.1) ocaml/opam-repository#24288

Merged

nberth pushed a commit to nberth/opam-repository that referenced this pull request


          [new release] rtree (0.1.1)

5b6f9c2

CHANGES:

## Bugs, Fixes and Optimisations

 - Remove extra length calculations (geocaml/ocaml-rtree#16, @lindig)
 - Remove some polymorphic comparisons and replace with `Float` functions (geocaml/ocaml-rtree#15, @patricoferris)
 - Fix stack overflows in OMT and Rectangle.merge (geocaml/ocaml-rtree#14, @patricoferris, reported by @lindig)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet