<div dir="ltr">Also my callsign is VA3IHX not W2OOT ;)<div><div><br></div><div>+# And thanks to Ian Harris (W2OOT) for decoding the mamory map.  <br></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, May 31, 2019 at 1:34 PM Dan Smith via chirp_devel &lt;<a href="mailto:chirp_devel@intrepid.danplanet.com">chirp_devel@intrepid.danplanet.com</a>&gt; wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">&gt; # HG changeset patch<br>
&gt; # User Rick DeWitt &lt;<a href="mailto:aa0rd@yahoo.com" target="_blank">aa0rd@yahoo.com</a>&gt;<br>
&gt; # Date 1559226735 25200<br>
&gt; #      Thu May 30 07:32:15 2019 -0700<br>
&gt; # Node ID 120e390016146e92f490619e4f7029ae60418d72<br>
&gt; # Parent  76c88493bd3f873d355d08e66c624c6acf201b4d<br>
&gt; [th-uv8000] New driver for TYT TH-UV8000 family, fixes issue #2837<br>
&gt; <br>
&gt; Submitting new driver in support of 11 issues 2837 ... 6183<br>
<br>
I didn&#39;t check them all, but I don&#39;t see a test image in the patch or in issue #2837. Can you attach one there so I can run tests?<br>
<br>
Also, you need to add the file to tools/cpep8.manifest and run the style checks. There are 76 errors when I do that and run &quot;tox -estyle&quot; locally.<br>
<br>
&gt; diff -r 76c88493bd3f -r 120e39001614 chirp/drivers/th_uv8000.py<br>
&gt; --- /dev/null Thu Jan 01 00:00:00 1970 +0000<br>
&gt; +++ b/chirp/drivers/th_uv8000.py      Thu May 30 07:32:15 2019 -0700<br>
&gt; @@ -0,0 +1,1598 @@<br>
&gt; +# Copyright 2019: Rick DeWitt (RJD), &lt;<a href="mailto:aa0rd@yahoo.com" target="_blank">aa0rd@yahoo.com</a>&gt;<br>
&gt; +# Version 1.0 for TYT-UV8000D/E<br>
&gt; +# Thanks to Damon Schaefer (K9CQB)and the Loudoun County, VA ARES <br>
&gt; +#    club for the donated radio.<br>
&gt; +# And thanks to Ian Harris (W2OOT) for decoding the mamory map.<br>
<br>
Unless I&#39;m misunderstanding what went on, I think &quot;memory&quot; is misspelled above :)<br>
<br>
&gt; +def _clean_buffer(radio):<br>
&gt; +    radio.pipe.timeout = 0.005<br>
&gt; +    junk = radio.pipe.read(256)<br>
&gt; +    radio.pipe.timeout = STIMEOUT<br>
&gt; +    if junk:<br>
&gt; +        Log.debug(&quot;Got %i bytes of junk before starting&quot; % len(junk))<br>
<br>
This line will fail with NameError, I think, as &quot;Log&quot; shouldn&#39;t be defined.<br>
<br>
&gt; +def _recv(radio, addr, length):<br>
&gt; +    &quot;&quot;&quot;Get data from the radio &quot;&quot;&quot;<br>
&gt; +<br>
&gt; +    data = _rawrecv(radio, length)<br>
&gt; +<br>
&gt; +    # DEBUG<br>
&gt; +    LOG.info(&quot;Response:&quot;)<br>
&gt; +    LOG.debug(util.hexprint(data))<br>
<br>
This doesn&#39;t make sense...to log a prefix at one level and the actual info at another. This should all be at debug level, and all in one log statement.<br>
<br>
I&#39;ll look in more detail when you&#39;ve got the style changes done and I can run tests against an image.<br>
<br>
Thanks!<br>
<br>
--Dan<br>
_______________________________________________<br>
chirp_devel mailing list<br>
<a href="mailto:chirp_devel@intrepid.danplanet.com" target="_blank">chirp_devel@intrepid.danplanet.com</a><br>
<a href="http://intrepid.danplanet.com/mailman/listinfo/chirp_devel" rel="noreferrer" target="_blank">http://intrepid.danplanet.com/mailman/listinfo/chirp_devel</a><br>
Developer docs: <a href="http://chirp.danplanet.com/projects/chirp/wiki/Developers" rel="noreferrer" target="_blank">http://chirp.danplanet.com/projects/chirp/wiki/Developers</a><br>
</blockquote></div>