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

Add Subway demo #1397

Open
wants to merge 447 commits into
base: master
Choose a base branch
from
Open

Add Subway demo #1397

wants to merge 447 commits into from

Conversation

mqcmd196
Copy link
Member

YutoUchimi and others added 30 commits November 28, 2023 11:59
@mqcmd196
Copy link
Member Author

To test it in pr1040

Setup

  1. Clone the repository and build it in your workspace
  2. Setup SSH
ssh-keygen
ssh-keygen -p -m PEM -f ~/.ssh/id_rsa # Without it, pr2 cannot ssh c2 when launch

Run

  1. Set environment variables (It is not required when you launch in applications user and launched with app_manager)
source /var/lib/robot/config.bash
export ROS_ENV_LOADER=$HOME/ros/demo_ws/devel/env.sh
  1. Launch
roslaunch elevator_move_base_pr2 elevator_move_base_eng2.launch
roslaunch elevator_move_base_pr2 fetch_sandwich.launch

@tkmtnt7000
Copy link
Member

tkmtnt7000 commented Dec 7, 2023

We may have faced the same problems as the following issue and pull request.
screen-point doesn't match with the target elevator button.
brightness

#1097
euslisp/jskeus#287 (comment)

@tkmtnt7000
Copy link
Member

tkmtnt7000 commented Dec 7, 2023

@tkmtnt7000
Copy link
Member

tkmtnt7000 commented Dec 7, 2023

We may have faced the same problems as the following issue and pull request. screen-point doesn't match with the target elevator button. brightness

#1097 euslisp/jskeus#287 (comment)

We almost have won that issue thanks to @pazeshun.
We need :pr1040 option #1321 (comment)

  • before
    Screenshot from 2023-12-07 18-19-35

  • after
    Screenshot from 2023-12-07 19-12-11

@tkmtnt7000
Copy link
Member

I also fixed python2 to python3 compatibility mqcmd196#4 .

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.

4 participants