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

Calibrate Unit D Multisense, add automatic general setup to drc_environment.sh #95

Merged
merged 17 commits into from
Mar 5, 2016
Merged

Calibrate Unit D Multisense, add automatic general setup to drc_environment.sh #95

merged 17 commits into from
Mar 5, 2016

Conversation

wxmerkt
Copy link
Member

@wxmerkt wxmerkt commented Mar 1, 2016

  • Calibrated Multisense on Unit D
  • Added two Valkyrie robot.cfg files, one for unit D and one for unit C (presumably)
  • Added generalisation to drc_environment that will set up VAL_LINK_IP, VAL_ZELDA_IP, VAL_MULTISENSE_IP, and VAL_UNIT based on whether we are running at MIT or Edinburgh. These environment variables can then be used in robot.pmd and other files to avoid having to fork/duplicate files

OUTSIDE_HOSTNAME=`nslookup $OUTSIDE_IP`

EDINBURGH='.ed.ac.uk'
MIT='mit.edu'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lets revise this before mergin

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Revised as on the observer workstation, still need to pull onto operator workstation once reconnected to network. Ready to merge

@wxmerkt
Copy link
Member Author

wxmerkt commented Mar 5, 2016

This is cleaned up and should be ready to go. Incorporated our changes to the corresponding files from the ed-install-fixes branch.

@@ -114,6 +114,27 @@ setup_robot_computers()
fi
}

setup_valkyrie_computers()
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

setup_edinburgh_valkyrie_computers()

<<< I see now, no need to respond >>>

@mauricefallon
Copy link
Contributor

This looks fine by me. happy to accept. uses the same approach as Pat wrote for Valkyrie

wxmerkt pushed a commit that referenced this pull request Mar 5, 2016
Calibrate Unit D Multisense, add automatic general setup to drc_environment.sh
@wxmerkt wxmerkt merged commit 07406bf into openhumanoids:master Mar 5, 2016
@wxmerkt wxmerkt deleted the calibrate-unit-d-multisense branch March 5, 2016 18:16
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