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

steering angle in gym simulator #966

Merged
merged 2 commits into from
Nov 9, 2023
Merged

steering angle in gym simulator #966

merged 2 commits into from
Nov 9, 2023

Conversation

tajgr
Copy link
Collaborator

@tajgr tajgr commented Nov 7, 2023

This PR enables use steering angle beside angular speed in the Gym simulator. The simulator robot uses Ackerman steering. The name of the channel is "move" as well as for Spider.

@tajgr
Copy link
Collaborator Author

tajgr commented Nov 7, 2023

For tests run: python -m osgar.go run test-gym_simulation.json -a 30 in osgar/examples/gym

@tajgr tajgr requested a review from m3d November 7, 2023 19:03
@tajgr
Copy link
Collaborator Author

tajgr commented Nov 9, 2023

Yesterday we discussed the appropriateness of using the "move" command for control. move was used because it already exists in the case of the Spider robot. I don't think it is appropriate to have multiple control commands if they contain the same information (speed, steering angle). So, how to solve the point?

  • Keep move. Too general, may cause confusion in the future.
  • Rename appropriately. The change must also be made for Spider. This will affect other projects and make it difficult to replay older logs. But there aren't that many logs. It is solvable (probably better now than in the future).
  • What name to choose? I don't think it is appropriate to use the word "ackermann" because the robot does not need to use ackermann steering.

How about: desired_steering or desired_driving @m3d ??

@m3d
Copy link
Member

m3d commented Nov 9, 2023

as discussed yesterday "move" is too generic and for Spider it was meant to direct all wheels in given angle (no longer true with Car mode). I would like separate control - there used to be John Deere (one of the OSGAR driven machines), where you could also steer without need to move. Then there will be PatAMat (tomorrow?) so my motivation to standardize this is also higher ;).
I liked ROS topic (probably not used often)
http://docs.ros.org/en/melodic/api/ackermann_msgs/html/msg/AckermannDrive.html
... there is even steering angle velocity. Yet another "issue" is for odometry you need more like the base length etc.
My preference would be desired_steering if "desired" has to be part of control :). But feel free to leave it for the simulator (it will change in future anyway).

@tajgr
Copy link
Collaborator Author

tajgr commented Nov 9, 2023

Ok, we can discuss it later (next Tuesday?). Thanks.

@tajgr tajgr merged commit 4f47240 into master Nov 9, 2023
6 checks passed
@tajgr tajgr deleted the feature/simulation-steer branch November 9, 2023 20:49
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