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

ARDrone2 full error message unclear #967

Closed
kevindehecker opened this issue Nov 19, 2014 · 4 comments
Closed

ARDrone2 full error message unclear #967

kevindehecker opened this issue Nov 19, 2014 · 4 comments
Labels
Enhancement Improve upon exiting functionality Minor
Milestone

Comments

@kevindehecker
Copy link
Contributor

When the ARDrone's memory is full but the users attempts to upload, the following error message is presented:

raise error_temp, resp

ftplib.error_temp: 451 Error
make[1]: *** [upload] Error 1
make[1]: Leaving directory /home/houjebek/paparazzi/sw/airborne' make: *** [ap.upload] Error 2 make: Leaving directory/home/houjebek/paparazzi'

I'd guess this error changes according to the amount of space left beforehand. Anyway, very unclear.

@dewagter
Copy link
Member

*exception *ftplib.error_temp
https://docs.python.org/2/library/ftplib.html#ftplib.error_temp

Exception raised when an error code signifying a temporary error (response
codes in the range 400–499) is received.
4xxTransient Negative Completion reply

The command was not accepted and the requested action did not take place,
but the error condition is temporary and the action may be requested again.
The user should return to the beginning of the command sequence, if any. It
is difficult to assign a meaning to "transient", particularly when two
distinct sites (Server- and User-processes) have to agree on the
interpretation. Each reply in the 4xx category might have a slightly
different time value, but the intent is that the user-process is encouraged
to try again. A rule of thumb in determining if a reply fits into the 4xx
or the 5xx (Permanent Negative) category is that replies are 4xx if the
commands can be repeated without any change in command form or in
properties of the User or Server (e.g., the command is spelled the same
with the same arguments used; the user does not change his file access or
user name; the server does not put up a new implementation.)

@OpenUAS OpenUAS added this to the v5.6 milestone Nov 20, 2014
@OpenUAS OpenUAS added Acknowledged Minor Enhancement Improve upon exiting functionality and removed Acknowledged labels Nov 20, 2014
@dewagter
Copy link
Member

@kevindehecker does it look better when you
git cherry-pick c321a03
?

@OpenUAS OpenUAS changed the title ARDrone full error message unclear ARDrone2 full error message unclear Nov 21, 2014
@kevindehecker
Copy link
Contributor Author

No:

make[1]: Leaving directory /home/houjebek/paparazzi/sw/airborne' cd sw/airborne; make TARGET=ap upload make[1]: Entering directory/home/houjebek/paparazzi/sw/airborne'
Uploading 'cdc-acm.ko
insmod: can't insert '/data/video/cdc-acm.ko': invalid module format
Kill running ap.elf and make folder raw
Uploading 'ap.elf' from /home/houjebek/paparazzi/var/aircrafts/ardrone2_raw/ap to raw
Traceback (most recent call last):
File "/home/houjebek/paparazzi/sw/ext/ardrone2_drivers/ardrone2.py", line 432, in
uploadfile(args.folder + "/" + f[1], file(args.file, "rb"))
File "/home/houjebek/paparazzi/sw/ext/ardrone2_drivers/ardrone2.py", line 18, in uploadfile
except ftplib.error_temp:
NameError: global name 'ftplib' is not defined
make[1]: *** [upload] Error 1
make[1]: Leaving directory /home/houjebek/paparazzi/sw/airborne' make: *** [ap.upload] Error 2 make: Leaving directory/home/houjebek/paparazzi'

BTW, I also happened to have another memory full related error. In this case the drone was 100% full, (the previous case probably 99% orso)

Uploading 'ap.elf' from /home/houjebek/paparazzi/var/aircrafts/ardrone2_raw/ap to raw
Traceback (most recent call last):
File "/home/houjebek/paparazzi/sw/ext/ardrone2_drivers/ardrone2.py", line 422, in
ftp.storbinary("STOR " + args.folder + "/" + f[1], file(args.file, "rb"))
File "/usr/lib/python2.7/ftplib.py", line 475, in storbinary
conn.sendall(buf)
File "/usr/lib/python2.7/socket.py", line 224, in meth
return getattr(self._sock,name)(args)
socket.error: [Errno 104] Connection reset by peer
make[1]: *
* [upload] Error 1
make[1]: Leaving directory /home/houjebek/paparazzi/sw/airborne' make: *** [ap.upload] Error 2 make: Leaving directory/home/houjebek/paparazzi'
'

@dewagter
Copy link
Member

closed with c046fa7

dewagter added a commit to kevindehecker/paparazzi that referenced this issue Nov 24, 2014
@OpenUAS OpenUAS modified the milestones: v5.6, v5.4 Dec 12, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement Improve upon exiting functionality Minor
Projects
None yet
Development

No branches or pull requests

3 participants