[chirp_devel] [PATCH] [ft4] Add progress bar, better error messages [#6573]
# HG changeset patch # User DanClemmensen DanClemmensen@gmail.com # Date 1551987068 28800 # Thu Mar 07 11:31:08 2019 -0800 # Node ID 0942fe2563a1457e5ccaa99e9032c5e072ab7151 # Parent ba16deb4cf3823ce2f7b9ecd7015a640c4545f71 [ft4] Add progress bar, better error messages [#6573]
diff -r ba16deb4cf38 -r 0942fe2563a1 chirp/drivers/ft4.py --- a/chirp/drivers/ft4.py Tue Mar 05 10:35:57 2019 -0800 +++ b/chirp/drivers/ft4.py Thu Mar 07 11:31:08 2019 -0800 @@ -237,7 +237,7 @@ i += 1 if i > toolong: LOG.debug("Response too long. got" + util.hexprint(response)) - raise errors.RadioError("Response too long.") + raise errors.RadioError("Response from radio too long.") return(response)
@@ -273,14 +273,18 @@ return response
-def startcomms(radio): +def startcomms(radio, way): """ For either upload or download, put the radio into PROGRAM mode and check the radio's ID. In this preliminary version of the driver, the exact nature of the ID has been inferred from a single test case. + set up the progress bar send "PROGRAM" to command the radio into clone mode read the initial string (version?) """ + progressbar = chirp_common.Status() + progressbar.msg = "Cloning " + way + " radio" + progressbar.max = radio.numblocks if b"QX" != sendcmd(radio.pipe, b"PROGRAM", 2): raise errors.RadioError("expected QX from radio.") id_response = sendcmd(radio.pipe, b'\x02', None) @@ -290,11 +294,12 @@ msg = "ID mismatch. Expected" + util.hexprint(radio.id_str) msg += ", Received:" + util.hexprint(id_response) LOG.warning(msg) - raise errors.RadioError("Incorrect ID.") + raise errors.RadioError("Incorrect ID read from radio.") else: msg = "ID suspect. Expected" + util.hexprint(radio.id_str) msg += ", Received:" + util.hexprint(id_response) LOG.warning(msg) + return progressbar
def getblock(pipe, addr, image): @@ -312,7 +317,7 @@ raise errors.RadioError("Incorrect response to read.") if checkSum8(response[1:20]) != bytearray(response)[20]: LOG.debug(b"Bad checksum: " + util.hexprint(response)) - raise errors.RadioError("bad block checksum.") + raise errors.RadioError("bad block checksum on read from radio.") image[addr:addr+16] = response[4:20]
@@ -326,9 +331,11 @@ """ image = bytearray(radio.get_memsize()) pipe = radio.pipe # Get the serial port connection - startcomms(radio) + progressbar = startcomms(radio, "from") for _i in range(radio.numblocks): getblock(pipe, 16 * _i, image) + progressbar.cur = _i + radio.status_fn(progressbar) sendcmd(pipe, b"END", 0) return memmap.MemoryMap(bytes(image))
@@ -351,10 +358,12 @@ send "END" """ pipe = radio.pipe # Get the serial port connection - startcomms(radio) + progressbar = startcomms(radio, "to") data = get_mmap_data(radio) for _i in range(1, radio.numblocks): putblock(pipe, 16*_i, data[16*_i:16*(_i+1)]) + progressbar.cur = _i + radio.status_fn(progressbar) sendcmd(pipe, b"END", 0) return # End serial transfer utilities diff -r ba16deb4cf38 -r 0942fe2563a1 tests/run_tests --- a/tests/run_tests Tue Mar 05 10:35:57 2019 -0800 +++ b/tests/run_tests Thu Mar 07 11:31:08 2019 -0800 @@ -1,3 +1,3 @@ #!/bin/bash
-exec python $(readlink -f $0).py $* +exec python2 $(readlink -f $0).py $*
diff -r ba16deb4cf38 -r 0942fe2563a1 tests/run_tests --- a/tests/run_tests Tue Mar 05 10:35:57 2019 -0800 +++ b/tests/run_tests Thu Mar 07 11:31:08 2019 -0800 @@ -1,3 +1,3 @@ #!/bin/bash
-exec python $(readlink -f $0).py $* +exec python2 $(readlink -f $0).py $*
I'm assuming you did not mean to include this in your change... ;)
--Dan
Oops. Sorry. It would be nice for me personally, but no, I don't guess we really want that in the daily build :-)
On Fri, Mar 8, 2019 at 7:38 AM Dan Smith dsmith@danplanet.com wrote:
diff -r ba16deb4cf38 -r 0942fe2563a1 tests/run_tests --- a/tests/run_tests Tue Mar 05 10:35:57 2019 -0800 +++ b/tests/run_tests Thu Mar 07 11:31:08 2019 -0800 @@ -1,3 +1,3 @@ #!/bin/bash
-exec python $(readlink -f $0).py $* +exec python2 $(readlink -f $0).py $*
I'm assuming you did not mean to include this in your change... ;)
--Dan
participants (3)
-
Dan Clemmensen
-
Dan Smith
-
DanClemmensen