Hi Nathan,
This is my first submission to the project. Hopefully I'm following the process well enough. I tested this driver fairly thoroughly with the radio when I had it, but no longer have access to it.
Cool, thanks!
Wasn't sure how to include the binary file for unit tests, which I ran. I attached it to this email.
Just attaching it here is okay. It would be even better to attach it to the ticket, because it doesn't get sent to all the subscribers and it's kind of archived with the request.
# HG changeset patch # User Nathan Crapo nathan@n4nc3o.com # Date 1465334892 21600 # Tue Jun 07 15:28:12 2016 -0600 # Node ID 2f356864c55f674a6faf157d7e13868e5275cf72 # Parent 333a280ca0c4e856258ebf9dfdb7c547fa9ec90c Adding support for TYT TH-7800. Fixes #3477.
diff -r 333a280ca0c4 -r 2f356864c55f chirp/drivers/th7800.py --- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/chirp/drivers/th7800.py Tue Jun 07 15:28:12 2016 -0600 @@ -0,0 +1,780 @@ +# Copyright 2014 Tom Hayward tom@tomh.us +# Copyright 2014 Jens Jensen af5mi@yahoo.com +# Copyright 2014 James Lee N1DDK jml@jmlzone.com +# Copyright 2016 Nathan Crapo nathan@n4nc3o.com (TH-7800 only)
Is this a near copy of another radio? Is it significantly different such that making them share a common base is not feasible?
+# --------------- Common Code ---------------
+# This section should go somewhere common like settings.py. Keep it here for +# now until other developers review and accept or reject it. +class RadioSettingValueMap(RadioSettingValueList):
Hmm, why is this copied into the driver?
add_radio_setting(basic, "apo", "Auto Power off (Hours)",[("Off", 0), ("0.5", 5), ("1.0", 10), ("1.5", 15), ("2.0", 20)],_settings.apo)
Can you break these lines before 80 columns? There is a pep8 tester in the tree (ideally, just run run_all_tests.sh) to validate the style.
I didn't apply this to run the tests, but I assume they all pass with this?
Thanks!
--Dan