Skip to content

Ignormals #48

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

Merged
merged 34 commits into from
Aug 5, 2017
Merged

Ignormals #48

merged 34 commits into from
Aug 5, 2017

Conversation

hexus
Copy link
Owner

@hexus hexus commented Aug 5, 2017

Replaced heuristics with a smarter (still slightly heuristic) approach to avoiding unwanted collisions for AABBs. Customised functions from SAT.js in order to achieve this.

Implemented debug rendering for tile normals and ignormals.

Improved performance by way of pooling arrays, response objects, vectors, and so on.

Removed the snap feature.

hexus and others added 29 commits May 19, 2017 02:14
#40).

Removed some code that will be replaced by this approach.
This is much closer to the desired result, but there are still some issues to overcome.
…is (#40).

There are still some odditties to be worked out, but it's nice to be this close.
Improved memory consumption in TileSlopeFactory.flagInternalEdges() (#42).
Larger-than-or-equal check for axis separation, instead of equal - this ignores touching shapes (#40).
…of another approach.

Improved some documentation and bail conditions for some collision methods.
).

This works. It actually works! 😭 Excellent.
…retty code (#40, #42).

Fixed small polygons colliding from inside internal tile groups (#40).
Minor update to .gitignore for npm-debug.log files.
Maybe they need a better name now...
So nice to see this one checked off already.
When collisions were tested against multiple tiles, each test would reset the response regardless of a collision. Now this only happens if there was indeed collision.
@hexus hexus self-assigned this Aug 5, 2017
@hexus hexus merged commit 190f6fe into master Aug 5, 2017
@hexus hexus deleted the ignormals branch August 5, 2017 11:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant