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.

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

For real concept 2017112701 #224

Merged
merged 4 commits into from
Nov 27, 2017
Merged

Conversation

louis-langholtz
Copy link
Owner

Description - What's this PR do?

  • Replaces duplicated code in favor of more generic code using templates.
  • Gets more specific about what's available functionality from templates.
  • Adds to the unit test coverage for UnitVec2 and TOI related code.
  • Fixes bug in corner case detection code.
  • Changes related to handling larger scales and better understanding of numerical limitations.

Related Issues

Issue #217.

@louis-langholtz louis-langholtz added the Enhancement For suggestions or changes that enhance any part of the project and isn't a bug. label Nov 27, 2017
@louis-langholtz louis-langholtz added this to the Beta Launch milestone Nov 27, 2017
@louis-langholtz louis-langholtz self-assigned this Nov 27, 2017
@coveralls
Copy link

coveralls commented Nov 27, 2017

Coverage Status

Coverage increased (+0.07%) to 95.617% when pulling 46df3fd on for-real-concept-2017112701 into 6ddc006 on master.

@louis-langholtz louis-langholtz merged commit c5c6d81 into master Nov 27, 2017
@louis-langholtz louis-langholtz deleted the for-real-concept-2017112701 branch November 27, 2017 20:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement For suggestions or changes that enhance any part of the project and isn't a bug.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants