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

gz-sim: detachable_joint.sdf #238

Closed
1 of 2 tasks
azeey opened this issue Aug 31, 2023 · 8 comments
Closed
1 of 2 tasks

gz-sim: detachable_joint.sdf #238

azeey opened this issue Aug 31, 2023 · 8 comments

Comments

@azeey
Copy link
Contributor

azeey commented Aug 31, 2023

Setup

  • Os: Ubuntu Jammy
  • BuildType: Binary

Links

Checks

  • Instructions work
  • Images (if there are any) match the result

You can find the code used to generate this test case here

@JACOBIN-SCTCS
Copy link

The state of the breadcrumb isn't getting monitored properly for this simulation. Only after I did a reset did the state of a breadcrumb appeared. Was this the expected behaviour (Attaching video). Can somebody also verify too.

detachable_joint-2023-09-07_21.49.58.mp4

@Voldivh
Copy link
Contributor

Voldivh commented Sep 7, 2023

I got the exact same behavior. Could you take a look at it @ahcorde ?

@iche033
Copy link

iche033 commented Sep 7, 2023

I think msgs are published to the state topic only if there is a change in state (attached -> detached or vice versa) as opposed to periodic updates.

Regarding reset causing a msg to be published on reset, that probably revealed an issue. It seems like an extra detachable joint is created whenever reset is invoked. It could be a bug.

@Voldivh
Copy link
Contributor

Voldivh commented Sep 7, 2023

I think msgs are published to the state topic only if there is a change in state (attached -> detached or vice versa) as opposed to periodic updates.

Yep, just tried it out and works as you explained. Thanks.

Regarding reset causing a msg to be published on reset, that probably revealed an issue. It seems like an extra detachable joint is created whenever reset is invoked. It could be a bug.

After playing a little bit around with it, the behavior does seems weird. I'm not sure if I understood what you mean by "an extra detachable joint is created...". The msg I received is related to the joints that were part of the simulation before:
Video

@iche033
Copy link

iche033 commented Sep 7, 2023

I was looking at the terminal that's running gz sim. Everytime I press reset, I see the msg:

[Dbg] [Physics.cc:1864] Creating detachable joint [53]

The entity number ([53] in the above example) keeps increasing so I assumed that new joint entities are created.

@Voldivh
Copy link
Contributor

Voldivh commented Sep 7, 2023

Oh, I'm getting something different when I click reset:

Msg [NameManager::issueNewName] (Skeleton::Joint | vehicle_blue) The name [fixed] is a duplicate, so it has been renamed to [fixed(1)]
Msg [NameManager::issueNewName] (Skeleton::Joint | vehicle_blue) The name [fixed] is a duplicate, so it has been renamed to [fixed(2)]

@ahcorde
Copy link

ahcorde commented Sep 8, 2023

can you try this patch ?

gazebosim/gz-sim#2131

This should reattach the models again

@azeey
Copy link
Contributor Author

azeey commented Sep 25, 2023

Closing since PR has been opened

@azeey azeey closed this as completed Sep 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants