<div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote">On 23 March 2018 at 22:38, Dan Smith via chirp_devel <span dir="ltr"><<a href="mailto:chirp_devel@intrepid.danplanet.com" target="_blank">chirp_devel@intrepid.danplanet.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">> - _memsize = 65227 # 65227 read from dump ?<br>
> - _block_lengths = [10, 65555] # ????? Not sure why this works to match _memsize<br>
> + _memsize = 65227 # 65227 read from dump<br>
> + _block_lengths = [10, 65217]<br>
<br>
</span>I was going to complain that you just tweaked the comment on the memsize line unnecessarily, but... are you removing that "?" because you really weren't sure before and now you've got it figured out?<br>
<br></blockquote><div><br></div><div>Yep.. </div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
And, are you saying that you're confident now that 65217 is the right value for the last block after the yaesu_clone change? Like, 65555 was the magic value that worked for your own timing, with the zero-block reads you were getting before?<br>
<br></blockquote><div><br></div><div>Yep.. Mem dump is 65227, first 10 then 65216 then strictly the 1 for the checksum, but I included that which seems to work fine. 65555 read enough "extra blocks" to make up for the ones we were losing.. at least on Mac(!) From my first view of the FT1 I was concern, in the back of my mind why the memory size and block size did not match up.. </div><div> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I assume this is confirmed to fix the problem the other person was having?<br></blockquote><div><br></div><div>I have reproduced the error on another Windows machine, although mine worked(!)</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
If all the above is correct, then...cool :)<br></blockquote><div><br></div><div>Cheers!</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
> diff -r 96bc56916c95 -r 9e3b0ff76308 chirp/drivers/yaesu_clone.py<br>
> --- a/chirp/drivers/yaesu_clone.py Tue Mar 20 21:38:07 2018 -0400<br>
> +++ b/chirp/drivers/yaesu_clone.py Fri Mar 23 17:52:01 2018 +0000<br>
<br>
</span>So, this code hasn't changed in so long I'm a little nervous about changing it. Is there anyone else on the dev list here that can give it a sniff test? I'm not sure I have any yaesu radios I can easily test with, but I'll look.<br></blockquote><div><br></div><div>Completely agree. I would really like to see other people test this.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
If not, we'll just have to merge it and warn the users list to test it ASAP.<br>
<br></blockquote><div><br></div><div> Agreed</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
--Dan<br>
______________________________<wbr>_________________<br>
chirp_devel mailing list<br>
<a href="mailto:chirp_devel@intrepid.danplanet.com">chirp_devel@intrepid.<wbr>danplanet.com</a><br>
<a href="http://intrepid.danplanet.com/mailman/listinfo/chirp_devel" rel="noreferrer" target="_blank">http://intrepid.danplanet.com/<wbr>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/<wbr>projects/chirp/wiki/Developers</a><br>
</blockquote></div><br></div></div>