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

Fix parameter names in components.rst #959

Merged

Conversation

NoahMollerstuen
Copy link
Contributor

In the docs for image_proc::CropDecimateNode , change the parameter names x_offset and y_offset to offset_x and offset_y, matching the actual names of parameters defined in crop_decimate.cpp.

@mikeferguson
Copy link
Member

Thanks for catching that!

CI is going to fail due to rolling transition - but this doesn't change code, so merging.

@mikeferguson mikeferguson merged commit 58099ef into ros-perception:rolling Mar 8, 2024
1 of 3 checks passed
@NoahMollerstuen NoahMollerstuen deleted the offset-x-docs-fix branch March 8, 2024 04:05
@NoahMollerstuen
Copy link
Contributor Author

Wow, thanks for the quick review!

@mikeferguson
Copy link
Member

Wow, thanks for the quick review!

You got lucky, I typically do one or two checks a day on GitHub notifications, and this came through just as I was doing that check.

Kotochleb pushed a commit to Kotochleb/image_pipeline that referenced this pull request May 27, 2024
In the docs for `image_proc::CropDecimateNode` , change the parameter
names `x_offset` and `y_offset` to `offset_x` and `offset_y`, matching
the actual names of parameters defined in crop_decimate.cpp
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