request to change the name of the variable step for sensor_msgs/Image to widthStep to avoid confusion (Bug #1398)
| Status: | Cancelled | Start date: | |
|---|---|---|---|
| Priority: | Normal | Due date: | |
| Assignee: |  Alexander Shishkov |
% Done: | 0% |
| Category: | documentation | ||
| Target version: | 2.4.0 | ||
| Affected version: | Operating System: | ||
| Difficulty: | HW Platform: | ||
| Pull request: |
Description
Hi,
I would like to make a point regarding the variable name step which stores the size of a row in bytes which is same as saying no of columns in many cases. But it is not evidently clear by the name of this variable unless someone is familiar with its use in opencv as stepWidth which seems less ambiguous.
Another solution is to provide some documentation for such variables as part of documentation for sensor_msgs/Image. Otherwise it is very difficult to interpret what is the physical significance of this variable.
Associated revisions
Merge pull request #1398 from ilya-lavrenov:ocl_perf_gemm
History
Updated by Alexander Shishkov about 13 years ago
Thank you for the report!
Due to backward compatibility we can't change name of the field in OpenCV.
Also the field is well documented(http://www.ros.org/doc/api/sensor_msgs/html/msg/Image.html) in ROS docs.
Please contact the developers of ROS.
- Status changed from Open to Cancelled
Updated by Andrey Kamaev almost 13 years ago
- Target version set to 2.4.0
- Assignee set to Alexander Shishkov