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

Modularize Preprocessing Pipeline #295

Open
3 tasks
nachovizzo opened this issue Mar 18, 2024 · 0 comments · May be fixed by #412
Open
3 tasks

Modularize Preprocessing Pipeline #295

nachovizzo opened this issue Mar 18, 2024 · 0 comments · May be fixed by #412
Assignees
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@nachovizzo
Copy link
Collaborator

nachovizzo commented Mar 18, 2024

As exposed in #290 where we made the registration module a class, I think we should go ahead and do the same with the preprocessing module, allowing us for more customization/parametrization of this bit. I'm copying the last part of that PR in here:

From a system point of view, it's nice that we now have this design:

// KISS-ICP pipeline modules
Registration registration_;
VoxelHashMap local_map_;
AdaptiveThreshold adaptive_threshold_;

Makes me wonder if we shouldn't merge Deskew into Preprocessing and convert it to another component as well 👀 , similar to how we do it in Python 🤔 , but to be discussed later

Items to address

  • Deskew is a preprocessing step, should be there
  • I'd love to move the preprocessing step for Kitti out of our CORE library and just drop it in the Python bindings to avoid people reporting the wrong number in papers
  • Decide if it's worth doing the deskewing with tbb, probably we should benchmark it (I think the answer is yes). If we keep it, we should decide on a clean way to control the number of threads "globally" Add support for limiting num_threads in tbb task #252 (comment)

CC @tizianoGuadagnino , @benemer

@nachovizzo nachovizzo self-assigned this Mar 18, 2024
@tizianoGuadagnino tizianoGuadagnino self-assigned this Apr 4, 2024
@tizianoGuadagnino tizianoGuadagnino added enhancement New feature or request help wanted Extra attention is needed labels Jul 9, 2024
@tizianoGuadagnino tizianoGuadagnino linked a pull request Dec 5, 2024 that will close this issue
@tizianoGuadagnino tizianoGuadagnino linked a pull request Dec 5, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants