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

Update startsitl.sh #10

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

shubham-shahh
Copy link

Updated sim_vehicle.py path

Updated sim_vehicle.py path
@ericjohnson97
Copy link
Contributor

I'm not sure this is the best way to do it now. Thank you for pushing me to get this updated, but I think the desired way to use sitl now is to make a folder for each vehicle you are simulating and run the command from there. something like

mkdir drone1 
sim_vehicle.py -v ArduCopter -f gazebo-iris --console

it shouldn't be required to cd to a specific directory anymore since 4.0 now puts sim_vehicle.py on the path. This is added to the path when running the new install pre_reqs script https://ardupilot.org/dev/docs/building-setup-linux.html#install-some-required-packages

I need to sit down and think about this some more since this kinda breaks my tutorials.

@shubham-shahh
Copy link
Author

I'm not sure this is the best way to do it now. Thank you for pushing me to get this updated, but I think the desired way to use sitl now is to make a folder for each vehicle you are simulating and run the command from there. something like

mkdir drone1 
sim_vehicle.py -v ArduCopter -f gazebo-iris --console

it shouldn't be required to cd to a specific directory anymore since 4.0 now puts sim_vehicle.py on the path. This is added to the path when running the new install pre_reqs script https://ardupilot.org/dev/docs/building-setup-linux.html#install-some-required-packages

I need to sit down and think about this some more since this kinda breaks my tutorials.

Hi, I have been to the page you mentioned but the sim_vehile.py wasn't added to the path, nor mavproxy.py was installed with that script. I think it is still under maintenance. For the time being, we should update the path in the .sh file for the people who are just getting started.

Thanks for making these awesome tutorials, I'll be more than happy to contribute to this project. Do let me know if I can help with something.

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