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.

Sign up for GitHub

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

Jump to bottom

Wheel size does not match description/controller #93

Closed
BarceloChristian opened this issue Jul 6, 2023 · 5 comments
Closed

Wheel size does not match description/controller #93

BarceloChristian opened this issue Jul 6, 2023 · 5 comments
Labels
bug Something isn't working

Comments

@BarceloChristian
Copy link
Collaborator

BarceloChristian commented Jul 6, 2023 •
edited
Loading

Environment

  • OS Version:
  • Source or binary build?

Description

  • Expected behavior:
    • Wheel size is defined the same across all the stack
  • Actual behavior:
    • Wheel Diameter is defined to 0.035 for Gazebo's diff drive plugin (here)
    • Wheel Radius is defined to 0.0331 (here)
@BarceloChristian BarceloChristian added the bug Something isn't working label Jul 6, 2023
@github-project-automation github-project-automation bot moved this to 🆕 Backlog in Andino Roadmap Jul 6, 2023
@elector102
Copy link
Collaborator

@BarceloChristian @francocipollone what is the real value?

@francocipollone
Copy link
Collaborator

@BarceloChristian @francocipollone what is the real value?

The real value of the Diameter is 6.6 cm

@agalbachicar
Copy link
Member

@JesusSilvaUtrera will take care right away of this task as he gets to know the platform and this seems to be great issue to begin with.

@JesusSilvaUtrera
Copy link
Contributor

JesusSilvaUtrera commented Mar 20, 2024 •
edited
Loading

I have created this PR for this issue @agalbachicar

jballoffet pushed a commit that referenced this issue Mar 20, 2024
@JesusSilvaUtrera
Fixed wheel diameter in URDF (#93) (#230)
dfa1378 
Signed-off-by: JesusSilvaUtrera <[email protected]>
@francocipollone
Copy link
Collaborator

Closed as completed at #230

@github-project-automation github-project-automation bot moved this from 🆕 Backlog to ✅ Done in Andino Roadmap Mar 20, 2024
jballoffet added a commit that referenced this issue Jun 17, 2024
@JesusSilvaUtrera @francocipollone
@garyservin @jballoffet
Jesus/#169 camera info yaml (#231)
8aa4aaa 
* Improves andino_navigation package. (#224)

Signed-off-by: Franco Cipollone <[email protected]>
Signed-off-by: JesusSilvaUtrera <[email protected]>

* Add basic Platform IO instructions (#229)

Signed-off-by: Gary Servin <[email protected]>
Signed-off-by: JesusSilvaUtrera <[email protected]>

* Modify Shell class API to allow dependency injection (#221)

Signed-off-by: Javier Balloffet <[email protected]>
Signed-off-by: JesusSilvaUtrera <[email protected]>

* Fixed wheel diameter in URDF (#93)

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

* Added YAML file for default camera intrinsics (#169)

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

* Adds references to o3de and isaac_sim simulations. (#232)

Signed-off-by: Franco Cipollone <[email protected]>
Signed-off-by: JesusSilvaUtrera <[email protected]>

* Added README for andino_bringup (#233)

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

* Added binaries installation docs (#234)

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

* andino_apps package created and andino_navigation package updated (#235)

* andino_apps package created and andino_navigation package updated

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

* Minor changes from the PR

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

* Update andino_navigation README with changes from PR

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

* Add andino_apps package to ci.yaml

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

* Fixed minor issues from the PR

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

* Added 'andino_apps' to the general README

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

---------

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

* Minimal change to test launch file

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

* Added webcam.yaml file and intrinsics are now publishing

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

* Added raspicam.yaml with camera intrinsics and updated instructions in andino_hardware/README

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

* Changed the 'camera_info_url' to be relative path

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

---------

Signed-off-by: Franco Cipollone <[email protected]>
Signed-off-by: JesusSilvaUtrera <[email protected]>
Signed-off-by: Gary Servin <[email protected]>
Signed-off-by: Javier Balloffet <[email protected]>
Co-authored-by: Franco Cipollone <[email protected]>
Co-authored-by: Gary Servin <[email protected]>
Co-authored-by: Javier Balloffet <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
Andino Roadmap
Status: ✅ Done
Milestone
No milestone
Development

No branches or pull requests
5 participants
@agalbachicar @elector102 @BarceloChristian @francocipollone @JesusSilvaUtrera