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

Re-implement the "Extended" Tracker using the new traversal techniques & lambdas. #309

Open
6 of 16 tasks
Algomorph opened this issue Jan 8, 2021 · 0 comments
Open
6 of 16 tasks
Assignees
Labels
ease-of-maintenance Resolving this issue would significantly improve ease of maintenance and future development. enhancement

Comments

@Algomorph
Copy link
Owner

Algomorph commented Jan 8, 2021

The new tracker will stick to the "Engine" convention for now. We can tentatively call it DenseColorAndDepthRigidAlignmentEngine, but it might make sense to combine the different tracker variants in a single class in case the old "Extended" tracker actually re-uses large portions of code from the old "Depth" tracker and the old "Color" tracker.

  • Debug through ExtendedTracker_CPU
  • Debug through ExtendedTracker_CUDA
  • Debug through ColorTracker_CPU again
  • Debug through DepthTracker_CPU
  • Inspect whether and which portions of code are reused by Extended tracker from Depth Tracker and Color Tracker
  • Inspect what kind of hierarchies do all three store and whether they can be combined

Consequence of the latter:

  • Combine the hierarchy level classes into a single class using variadic macros to control which images it holds

At this point, we should be more or less clear on what the architecture should be (how to organize the data and the classes/structs).

The parameters will need to be revised to use serializable structures and enums:

  • Required enums
  • Required struct(s)

Stick to the old function interface for now, hence the following functions will need to be re-implemented and tested for CPU/GPU and PVA/VBH:

  • Constructor and supporting methods, destructor (the first BIG one)
  • CanKeepTracking
  • TrackCamera (the second BIG one)
  • Update Initial Pose
  • RequiresColourRendering
  • RequiresDepthReliability
  • RequiresPointCloudRendering
@Algomorph Algomorph self-assigned this Jan 8, 2021
@Algomorph Algomorph added ease-of-maintenance Resolving this issue would significantly improve ease of maintenance and future development. enhancement labels Jan 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ease-of-maintenance Resolving this issue would significantly improve ease of maintenance and future development. enhancement
Projects
None yet
Development

No branches or pull requests

1 participant