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

interPointCD / PointColDetector #475

Merged
merged 2 commits into from Dec 17, 2015

Conversation

Projects
None yet
2 participants
@ulteq
Contributor

ulteq commented Dec 10, 2015

Reduced the complexity of the inter truck collision code:

  • Only one interPointCD object per truck [instead of multiple objects]
  • Each truck is only being worked on by a single thread pool worker [instead of multiple workers]
  • One kdTree per truck [instead of one kdTree per worker thread]
  • Less points per kdTree [per truck bounding box checks]

Removed all dead code in the PointColDetector

@only-a-ptr

This comment has been minimized.

Show comment
Hide comment
@only-a-ptr

only-a-ptr Dec 8, 2015

Member

👍

Member

only-a-ptr commented on 0d576af Dec 8, 2015

👍

only-a-ptr added a commit that referenced this pull request Dec 17, 2015

Merge pull request #475 from ulteq/interPointCD_Test
Ulteq:
Reduced the complexity of the inter truck collision code:

    Only one interPointCD object per truck [instead of multiple objects]
    Each truck is only being worked on by a single thread pool worker [instead of multiple workers]
    One kdTree per truck [instead of one kdTree per worker thread]
    Less points per kdTree [per truck bounding box checks]

Removed all dead code in the PointColDetector

@only-a-ptr only-a-ptr merged commit 0f0e46e into RigsOfRods:master Dec 17, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@ulteq ulteq deleted the ulteq:interPointCD_Test branch Dec 19, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment