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

chore(nebula_node_container): fix, confirm and re-enable ring_outlier_filter #316

Merged

Conversation

mojomex
Copy link

@mojomex mojomex commented Sep 17, 2024

This PR re-enables the ring_outlier_filter in nebula_node_container.launch.py which was previously disabled in:

The module is working (= outputting pointclouds at 10 Hz and not crashing) on my Desktop with one OT128 and one QT128 connected, with the current pilot-auto.x2.

Here are the versions used:

…with the latest versions:

nebula: develop-eval@81afef9ec1cdc1dae9d39e08e916813baab1fd4c
transport_drivers: autowarefoundation/transport_drivers.git@8a93ae022604df8132bd65ab8a599532cf4c0a98
ros2_socketcan: /autowarefoundation/ros2_socketcan.git@2bbdba46ce4177e3076892aa99ede557b4f82446
@TomohitoAndo
Copy link
Contributor

TomohitoAndo commented Sep 17, 2024

@mojomex
Thank you!!
It seems I don't have access to nebula@81afef9 ? Or it actually doesn't exist? 🤔
image

@mojomex
Copy link
Author

mojomex commented Sep 17, 2024

@TomohitoAndo Sorry, the wrong ref was pasted above. I edited it to point to the correct ref 🙇‍♂️

Copy link
Contributor

@TomohitoAndo TomohitoAndo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM
thanks!

@TomohitoAndo TomohitoAndo merged commit 3a457bd into beta/x2_gen2/v0.29.0 Sep 17, 2024
10 of 11 checks passed
@TomohitoAndo TomohitoAndo deleted the beta/x2_gen2/v0.29.0-fix-ring-outlier branch September 17, 2024 23:25
@mojomex mojomex restored the beta/x2_gen2/v0.29.0-fix-ring-outlier branch September 18, 2024 05:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants