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

feat(common_sensor_launch): add distortion corrector param #247

Merged

Conversation

vividf
Copy link
Contributor

@vividf vividf commented Jun 27, 2024

PR: autowarefoundation/autoware.universe#7137 removes the default parameters from the distortion correction node and add the parameters file for the distortion node.

As the default parameters are removed, nebula_node_container.launch.py needs to load the parameters from yaml file.

test with

ros2 launch autoware_launch logging_simulator.launch.xml map_path:=$HOME/autoware_map/sample-rosbag vehicle_model:=jpntaxi sensor_model:=aip_xx1 vehicle_id:=2

@vividf vividf requested a review from badai-nguyen June 27, 2024 05:37
@vividf vividf self-assigned this Jun 27, 2024
@vividf vividf requested review from YoshiRi and knzo25 July 2, 2024 02:53
@vividf
Copy link
Contributor Author

vividf commented Jul 2, 2024

@knzo25
regarding to autowarefoundation/sample_sensor_kit_launch#93.
I think you would like me to use ParameterFile instead of the load_composable_node_param for this PR as well right?

@vividf
Copy link
Contributor Author

vividf commented Jul 2, 2024

@knzo25
For this one, I would like just to use the current method load_composable_node_param.
I will modify it to ParameterFile when I create another PR when all of the parameters are changed in pointlcoud_preprocessor.
Otherwise, there are a lot of additional tests for other nodes that I need to work on.

Copy link
Contributor

@YoshiRi YoshiRi left a comment

Choose a reason for hiding this comment

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

LGTM for code itself.

Copy link
Contributor

@knzo25 knzo25 left a comment

Choose a reason for hiding this comment

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

LGTM (just in case I tested it locally)

@vividf vividf merged commit 40322e3 into tier4:tier4/universe Jul 3, 2024
10 of 11 checks passed
knzo25 pushed a commit that referenced this pull request Sep 12, 2024
* feat: add distortion corrector param

Signed-off-by: vividf <[email protected]>

* ci(pre-commit): autofix

---------

Signed-off-by: vividf <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
TomohitoAndo added a commit that referenced this pull request Sep 13, 2024
* fix(lidar): update PTP settings for new PTP architecture

* fix(lidar): set FoVs to their desired (no added padding for correction) values as Nebula handles this now

* chore(lidar): change parameters and node name to nebula/develop versions

* fix(nebula_node_container): add multicast_ip parameter

* fix(nebula_node_container): add point_filters parameter

* hotfix(nebula_container_launch): ring outlier filter segfaults, disable it for now

* ci(pre-commit): autofix

* feat(common_sensor_launch): add distortion corrector param (#247)

* feat: add distortion corrector param

Signed-off-by: vividf <[email protected]>

* ci(pre-commit): autofix

---------

Signed-off-by: vividf <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

* chore: add parameter for distortion corrector

---------

Signed-off-by: vividf <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Yi-Hsiang Fang (Vivid) <[email protected]>
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.

3 participants