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

Tweak to how polygon orientation is determined #2137

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Tweak to how polygon orientation is determined #2137

wants to merge 7 commits into from

Commits on Aug 15, 2024

  1. attempt at re-implementing algorithm 

    maybe better explains the weird behavior with the 3rd vertex (vtx2)
    @Jaklyy
    Jaklyy committed Aug 15, 2024
    Configuration menu
    Copy the full SHA
    0a908dd View commit details
    Browse the repository at this point in the history

  2. vertex coords appear to be 27 bit signed ints 

    at least for determining the orientation?
    @Jaklyy
    Jaklyy committed Aug 15, 2024
    Configuration menu
    Copy the full SHA
    047dccd View commit details
    Browse the repository at this point in the history

Commits on Aug 16, 2024

  1. correct some errors 

    was not actually 27 bit oops
    @Jaklyy
    Jaklyy committed Aug 16, 2024
    Configuration menu
    Copy the full SHA
    57d1043 View commit details
    Browse the repository at this point in the history

  2. w is used on hw

    @Jaklyy
    Jaklyy committed Aug 16, 2024
    Configuration menu
    Copy the full SHA
    f8589d5 View commit details
    Browse the repository at this point in the history

Commits on Aug 17, 2024

  1. undo everything

    @Jaklyy
    Jaklyy committed Aug 17, 2024
    Configuration menu
    Copy the full SHA
    f79eb72 View commit details
    Browse the repository at this point in the history

  2. rework polygon orientation calcs 

    but this time actually correct
    @Jaklyy
    Jaklyy committed Aug 17, 2024
    Configuration menu
    Copy the full SHA
    4c9e16d View commit details
    Browse the repository at this point in the history

Commits on Aug 18, 2024

  1. vectors need to be 64 bit 

    or alternatively we might need to normalize them somehow?
    @Jaklyy
    Jaklyy committed Aug 18, 2024
    Configuration menu
    Copy the full SHA
    094af17 View commit details
    Browse the repository at this point in the history