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.

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

Performance optimisations #19

Merged
merged 5 commits into from Nov 30, 2017
Merged

Performance optimisations #19

merged 5 commits into from Nov 30, 2017

Commits on Nov 29, 2017

  1. helixUtils::intersectWithZPlane: faster way to get distance to origin

    @andresailer
    andresailer committed Nov 29, 2017
    Copy the full SHA
    54a5b5d View commit details
    Browse the repository at this point in the history

  2. intersections: optimise intersection constructors

    @andresailer
    andresailer committed Nov 29, 2017
    Copy the full SHA
    cde5df6 View commit details
    Browse the repository at this point in the history

  3. intersections: use contructor depending on number of intersections 

    do the correct allocation of space

store inverse of normalSquare to avoid division, use multiplication
    @andresailer
    andresailer committed Nov 29, 2017
    Copy the full SHA
    4ca60e7 View commit details
    Browse the repository at this point in the history

  4. helixUtils: multiplication instead of division

    @andresailer
    andresailer committed Nov 29, 2017
    Copy the full SHA
    104d336 View commit details
    Browse the repository at this point in the history

  5. helixUtils: take return value by const ref for optimisation

    @andresailer
    andresailer committed Nov 29, 2017
    Copy the full SHA
    9fe2833 View commit details
    Browse the repository at this point in the history