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

ROS2 frame tests #4

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

ROS2 frame tests #4

wants to merge 3 commits into from

Conversation

adamkrawczyk
Copy link

Prepare tests for ROS2 frame component

Adam Krawczyk added 3 commits April 28, 2023 15:44
Signed-off-by: Adam Krawczyk <[email protected]>
Signed-off-by: Adam Krawczyk <[email protected]>
Signed-off-by: Adam Krawczyk <[email protected]>

- Run the game (`ctrl + G`)

- Check the `tf_static` by running the command `ros2 topic echo /tf_static` This should print the transform between the two frames. Move the camera (WASD) and check if the transform is updated.
Copy link
Collaborator

Choose a reason for hiding this comment

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

What is the purpose of moving the camera at this point? Is there any frame attached to the camera?

from azlmbr.entity import EntityType

import subprocess
DEGREE_RADIAN_FACTOR = 0.0174533
Copy link
Collaborator

Choose a reason for hiding this comment

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

Convert deg to rad with math.radians()


if __name__ == "__main__":

typeNameList = ["ROS2 Camera Sensor", "ROS2 Frame"]
Copy link
Collaborator

Choose a reason for hiding this comment

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

Inconsistent naming convention. Use lower_camel_case for temp variables as previously.


## ROS2 Checking if the transform is correct


Copy link
Collaborator

Choose a reason for hiding this comment

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

Unnecessary blank line.

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