-
Notifications
You must be signed in to change notification settings - Fork 0
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
Ray collision #79
Ray collision #79
Conversation
Can you please go through and add comments explaining the logic? Also resolve conflicts when you get the chance to checks can run |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just some preliminary PR review. Will check the controller part later
It's been over a month since opening and over a week since last commit, is this close to completion? This needs to be finished. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add this controller to the master controller game loop logic. Also fails to run after pulling the latest dev
…into Ray-Collision
…yale_2022 into Ray-Collision
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Runs, looks good
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would still like to see more comments, but we need this so I'll approve it. Can we get documentation on how this is supposed to be used
No description provided.