ROS-Aerial / community

45 stars 13 forks source link

Run Aerostack2 Gazebo example #66

Closed botmayank closed 2 weeks ago

botmayank commented 2 months ago
botmayank commented 2 months ago

Teleop and mission both fail, arm and emergency stop works

Image Image

Also, dependency for keyboard teleop (pysimplegui) should be linked on Prerequisites section ideally. I think we should also add a T-Mux cheatsheet here. I forgot how to switch tabs in Tmux for example πŸ˜…

pip install pysimplegui==4.60

I'll consolidate fixes once I have the example running and raise a PR for AS2.

knmcguire commented 2 months ago

I don't know what @miferco97 preference is but it might be good to start an issue in the aerostack2 github issue tracker as well? Would be good for them to know that this is missing from their standard installation.

miferco97 commented 2 months ago

Hello ! This information is super useful for us, we encourage you to create an issue in our documentation (https://github.com/aerostack2/aerostack2.github.io) and link it with this issue just for keeping track of correcting this errors and improve our documentation with your comments. I will also ping @javilinos, who has been the responsible of documenting this part.

javilinos commented 2 months ago

Teleop and mission both fail, arm and emergency stop works

Image Image

Also, dependency for keyboard teleop (pysimplegui) should be linked on Prerequisites section ideally. I think we should also add a T-Mux cheatsheet here. I forgot how to switch tabs in Tmux for example πŸ˜…

pip install pysimplegui==4.60

I'll consolidate fixes once I have the example running and raise a PR for AS2.

Hi @botmayank @miferco97, dependency for keyboard teleop is linked in keyboard teleop, however, it is a good idea to link this part in the prerequisites page so thanks for the feedback. As for the example itself, we are consolidating gazebo integration into Aerostack2 with all the migration from ignition nomenclature to the new gazebo sim one. Today we made a fix that should make the simulator work again. However, with a binary install the example should work as everything there is in Ignition Fortress version. We will look into it. Regards.

javilinos commented 2 months ago

Hi @botmayank, could you please provide us with the terminal output of the tmux window nΒΊ1: platform ? We suspect the error might be generating there. Thanks.

botmayank commented 1 month ago

@miferco97 @knmcguire I was going to create one but it seems like there's an existing one that can be escalated. @javilinos I think the ticket is self explanatory but maybe we can add labels/bump up priority?

https://github.com/aerostack2/aerostack2.github.io/issues/38

@javilinos thanks for looking into this! Here are other screenshots from last week when the demo didn't work for me:

Not working: Image Image Image Image

@javilinos is there a need to restart after any dependency installation for AS2? Today as I booted up my machine and ran the examples, everything just works πŸ˜… I launched using ./launch_as2.bash -t and ran the mission using python3 mission.py in the relevant Tmux term and it just worked! After the mission I could use teleop as well. I even tried to directly run teleop instead of the mission by using stop.bash and then starting again and trying teleop directly, that too works.

The only reason for a difference I can think of is that when I first ran those examples last week I didn't have pyqtgui installed initially, and then after installing I didn't restart the machine. Is that needed?

Today's screenshots of example working πŸ˜„

Image Image Image Image Image Image Image

@miferco97 At this point I don't know if there's a need to open a separate ticket for the behaviour I observed last week since I'm not sure how to replicate it and all I have are the other screenshots I've shared above πŸ˜• Any suggestions?

miferco97 commented 3 weeks ago

Hi @botmayank, we have not been able to replicate this, so I think we should move on. We are aware that the install guide should explicitly include the pysimplegui dependency to avoid these bugs. If you could collect all the things that should be documented in more detail that would be great!!!

botmayank commented 2 weeks ago

@miferco97 sure, will do, I'll close this for now as we've decided to use GZ-Harmonic + ROS2 Humble as our base stack. I'm trying this out next: https://aerostack2.github.io/_03_aerial_platforms/_gazebo_simulation/index.html#installation