Skip to content

Add troubleshooting/restarting SITL sections #176

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

Closed

Conversation

hamishwillee
Copy link
Contributor

This adds some troubleshooting when using vagrant (it is possible for the image to corrupt) and also instructions on resuming/restarting).

In addition, it adds some minor fixes to rendering of paths (backslashes) and the bullet points.

@tcr3dr
Copy link
Contributor

tcr3dr commented Jul 7, 2015

At this point I'm comfortable floating dronekit-sitl-runner as a cross-platform alternative to the Vagrant sections.

This uses precompiled binaries on S3 that are generated in a separate CI step: https://github.com/tcr3dr/dronekit-sitl-builder Three versions (3.2, 3.3-rc5, and 3.4-dev) are supported and built for Windows, OS X, and Linux.

Basically, we should just outline what assurances are needed from that tool to use as a replacement for the Vagrant install (which I assume is less desirable in all circumstances).

@mrpollo
Copy link
Member

mrpollo commented Jul 7, 2015

@tcr3dr if you feel like dksr (dk-sitl-runner) is the way to go we should remove all traces of sitl vagrant from the docs, lets create a new issue for this, and if this issue is now made irrelevant by that lets close this

long live dronekit-sitl-runner!

@hamishwillee
Copy link
Contributor Author

@tcr3dr I can't get it to work on Windows (dronekit/dronekit-sitl#9).

I can't get UDP connection to work on Linux.

Before replacing the vagrant stuff I would hope that we could get this working on Plane and Rover (dronekit/dronekit-sitl#3).

Does this support "help" ie so people can get what versions of the binaries are available?

Does this automatically set up any outputs or other defaults - ie what if you do "dronekit-sitl copter-3.4-dev"

Do any other dependencies need to be added? Or are mavproxy et al pip dependencies?

P.S. Excellent job! This should make things much easier for users!

@hamishwillee
Copy link
Contributor Author

@tcr3dr It would be best to push this out now, while we resolve issues with the sitl runner. The work is already done and minor.

@@ -52,7 +52,7 @@ and Linux (though we recommend the native installations linked above).
#. Install SSH (Windows only - SSH is present by default on Linux/Mac OSX)

* Download and install `Git for Windows <https://git-scm.com/download/win>`_ (or another client that comes with SSH).
After installing you can locate the file using the command ``C:\where ssh`` (normally it is installed to **C:\Program Files (x86)\Git\bin\ssh.exe**
After installing you can locate the file using the command ``C:\where ssh`` (normally it is installed to **C:/Program Files (x86)/Git/bin/ssh.exe**
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm confused about this change, shouldn't they be backslashes?

Also I believe that can read where ssh without the c:\

@tcr3dr
Copy link
Contributor

tcr3dr commented Jul 16, 2015

@hamishwillee Just one comment on the change, otherwise I'll merge.

@hamishwillee
Copy link
Contributor Author

@tcr3dr You're right - fixed by 6686cc0 . Note the double slashes are required because the first one is treated as an escape key.

@hamishwillee
Copy link
Contributor Author

@tcr3dr @mrpollo I'm closing this. Essentially we need to get DK-sitl running, and given our priorities this will never get in before that happens

@hamishwillee hamishwillee deleted the tmp_sitl_troubleshooting branch October 12, 2015 23:46
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.

3 participants