-
Notifications
You must be signed in to change notification settings - Fork 680
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
Optimise MAV FTP upload to improve performance #1270
Comments
I'm able to upload using a similar setup (Copter on master, Pixhawk1, MAVProxy from install 1.8.66). It is quite slow - around 15-25kByte/sec. A 33Mb terrain file took ~33min - so 1Mb/min transfer rate. If you use |
I've changed the title to reflect that this is no longer considered a bug but rather an enhancement request. On the Nov 14th dev call we discussed with Tridge and discovered that MAVFTP's implementation (both within the GCSs and the flight code) has been optimised for downloading (e.g. from the autopilot->GCS) but not for uploading (e.g. GCS->autopilot). There were two suggestions for improving peformance:
I will still try @stephendade's suggestion to confirm that the FTP is working on windows even if it is very slow. |
@stephendade (and anyone else who is interested), I ran, "ftp status" during the transfer and saw what's below so it is making (slow) progress. |
We have a report in Copter-4.4 that uploading a 30MB terrain file using MAVFTP takes >30min (see report here)
As part of the investigation I attempted to upload a 30MB terrain file (produced using https://terrain.ardupilot.org/) and confirmed that indeed MP does take at least 30min (it only uploaded 1MB of the file in 5min). See MP issue here: ArduPilot/MissionPlanner#3229
To try and determine if MAVProxy was any faster (in the hopes that it would be thus confirming that the issue was in MP) I found that MAVProxy's "ftp" command simply doesn't work at least on windows.
I was running the latest MAVProxy (1.8.66)
The exact procedure was:
By the way, if I ran the same ftp command but neglected to add the "/" after the "APM" I got this message spamming my console.
The text was updated successfully, but these errors were encountered: