-
Notifications
You must be signed in to change notification settings - Fork 101
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
Dependency script #60
Conversation
@xabierolaz @SwapnilPande @varunag18 This PR hopefully will make it a lot easy to install and run GymFC. If you can take a look or even test out the new script since you all have recently be getting the environment setup it would be much appreciated. I tested it in a VirtualBox VM with Ubuntu 18.04 with 15GB harddrive, 2 CPUs and 4GB memory. |
Great, thanks @wil3 , I'm testing it today, will let you know! |
Hi Wil, Im sory but I dont have another machine apart from the one im currently working on, to test the dependency script, but I wil test it once I get a machine at my university. Just having a look at it.....it seems good to go. However, there are certain things I would like to point out that I feel should be more clear for a new user. In the ReadMe file of the i58-dep-script, when we ask the user to |
This script looks great and definitely simplifies installing dependencies. It ALMOST worked for me, but still had a few issues. Since you tried in a VM, I attempted installing in a clean ubuntu 18.04 docker container. The only problem that arose was that the gazebo install directory is set to I have a couple of ideas for this:
|
@SwapnilPande Gazebo recommends installing to /home/$USER/local so you can easily switch between the debian build and the build from source. Thanks for trying it in Docker, I'd eventually like to use the same script for the Travis docker so it would be good to have it work for both environments. I'll test it in the VM with the default location (and update gymfc.ini) and if it works as expected could you test it again in Docker? |
Hi @wil3, can confirm gymfc works greatly on a clean ubuntu 18.04 computer |
Thats gr8 @xabierolaz . I need to test this now, as Wil has suggested a problem with my existing installation. Could you provide your thoughts on the question part of this comment? |
Hi Wil, I did a fresh install using the dependency script and it worked like a breeze. My test scripts are running as expected. Thanks a lot. |
I made a number of improvements, updated documentation for /examples and fixed the sample_noise bug. @SwapnilPande I made the switch to install to /usr/local, tested it in Virtualbox could you give it a try in docker? |
@all-contributors add @SwapnilPande for ideas and test |
I've put up a pull request to add @SwapnilPande! 🎉 |
@all-contributors add xabierolaz for test |
I've put up a pull request to add @xabierolaz! 🎉 |
@all-contributors add varunag18 for test |
I've put up a pull request to add @varunag18! 🎉 |
Sorry for the slow response, I will test on docker first thing tomorrow morning! |
Installation went properly except for adding
When running the script in docker (as root), I get the following error:
After manually running these commands without sudo, the gymfc tests work perfectly. Since you suggest in your docs running the install_dependencies.sh script with sudo,, you should be able to remove it from the individual commands within the script, correct? With that small change, the script works great! |
@SwapnilPande awesome thanks for testing that, great catch. Yes I can just remove sudo there. Will make the change then merge this all in. Thanks everyone for the help. |
Closes: #39 |
Closes: #58
This PR makes installing the dependencies alot easier by just having to run a bash script.
Turns out installing
pip3 install .
vs in edit modepip3 install -e .
is quite different. The custom Gazebo messages can't be found unless LD_LIBRARY_PATH is modified and also gymfc.ini was missing. The included fixes allow for installing viapip3 install .
.