<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
<style type="text/css" style="display:none;"> P {margin-top:0;margin-bottom:0;} </style>
</head>
<body dir="ltr">
<div style="font-family: Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
Oh, yeah, I was following the doc sequentially and got to the mail part before I got to the patch guidelines, oops. Issue #7021</div>
<div id="appendonsend"></div>
<hr style="display:inline-block;width:98%" tabindex="-1">
<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" style="font-size:11pt" color="#000000"><b>From:</b> chirp_devel-bounces@intrepid.danplanet.com <chirp_devel-bounces@intrepid.danplanet.com> on behalf of Tony Fuller via chirp_devel <chirp_devel@intrepid.danplanet.com><br>
<b>Sent:</b> Tuesday, August 27, 2019 7:43 PM<br>
<b>To:</b> chirp_devel@intrepid.danplanet.com <chirp_devel@intrepid.danplanet.com><br>
<b>Subject:</b> Re: [chirp_devel] [PATCH] Fixing issue where KGUV8D Plus did not allow savinng memories with certain valid tuning steps</font>
<div> </div>
</div>
<style>
<!--
.x_EmailQuote
{margin-left:1pt;
padding-left:4pt;
border-left:#800000 2px solid}
-->
</style>
<div>
<div dir="auto" style="direction:ltr; margin:0; padding:0; font-family:sans-serif; font-size:11pt; color:black">
I stand corrected. I missed the "plus" in the filename. :) I'm guessing the only thing the patch needs is an issue but will go ahead and listen now instead of talking (and looking silly)<br>
<br>
</div>
<div dir="auto" style="direction:ltr; margin:0; padding:0; font-family:sans-serif; font-size:11pt; color:black">
Tony<br>
<br>
</div>
<div dir="auto" style="direction:ltr; margin:0; padding:0; font-family:sans-serif; font-size:11pt; color:black">
<span id="x_OutlookSignature">
<div dir="auto" style="direction:ltr; margin:0; padding:0; font-family:sans-serif; font-size:11pt; color:black">
Get <a href="https://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Faka.ms%2Fghei36&data=02%7C01%7C%7Cdfc9f42b6bf74a8c212608d72b618d1d%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637025570288536918&sdata=3vgMaJKBEGepbUy5QYePkJsEGTfGyUezmkK%2ByhtpvXA%3D&reserved=0" originalsrc="https://aka.ms/ghei36" shash="I7QX+XshfhvGE8xTVW6HOfrkS+m8Da8NMfsHYu1kTJ0DchRiCrPhY558zJoTqExpr0oCi2K1sffRsHOZ2UiaARxKJXqeudZKFmasEcbd3pRyxTAjd7nREnHKLWVtEvYGPyNPxpg5QQENEmyrlQ+EoVOJok26Tn+yXkNNLXqgckc=">
Outlook for Android</a></div>
</span><br>
</div>
<hr tabindex="-1" style="display:inline-block; width:98%">
<div id="x_divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" color="#000000" style="font-size:11pt"><b>From:</b> Tony Fuller<br>
<b>Sent:</b> Tuesday, August 27, 2019 9:38:13 PM<br>
<b>To:</b> chirp_devel@intrepid.danplanet.com <chirp_devel@intrepid.danplanet.com><br>
<b>Subject:</b> Re: [chirp_devel] [PATCH] Fixing issue where KGUV8D Plus did not allow savinng memories with certain valid tuning steps</font>
<div> </div>
</div>
<div>
<div>
<div dir="auto" style="direction:ltr; margin:0; padding:0; font-family:sans-serif; font-size:11pt; color:black">
It looks like something very similar was merged in about 5 months ago in <a href="https://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fchirp.danplanet.com%2Fprojects%2Fchirp%2Frepository%2Frevisions%2F2ae5aef78c2e&data=02%7C01%7C%7Cdfc9f42b6bf74a8c212608d72b618d1d%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637025570288536918&sdata=1DZfv2z9iJdKFXafw%2BJUekjCwcrpRG7kwNL8qmL%2BtRE%3D&reserved=0" originalsrc="https://chirp.danplanet.com/projects/chirp/repository/revisions/2ae5aef78c2e" shash="YrkGWsQsIcNodT0CZjZa9HgOjY4cD2Z5QPxQn6oaSm2X0yET+/GxcQRNaQ4oWr6K5UOJhWcI3DBvV2Ib2dnZGm5VOp6HVPKN6yKUMgtpcZYxpCTp+aBCH6BpTby0wRuhsNFBDwF/bkUA9fWfIYTjlyjUGONdbIBoxw2PZkYyjJs=">
https://chirp.danplanet.com/projects/chirp/repository/revisions/2ae5aef78c2e</a> but on line 467 (rather than 519) although the context (3 lines above and below) look quite similar.<br>
<br>
</div>
<div dir="auto" style="direction:ltr; margin:0; padding:0; font-family:sans-serif; font-size:11pt; color:black">
Maybe you just need to run hg pull? It's totally possible I missed something too.<br>
<br>
</div>
<div dir="auto" style="direction:ltr; margin:0; padding:0; font-family:sans-serif; font-size:11pt; color:black">
Tony<br>
<br>
</div>
<div dir="auto" style="direction:ltr; margin:0; padding:0; font-family:sans-serif; font-size:11pt; color:black">
<span id="x_x_OutlookSignature">
<div dir="auto" style="direction:ltr; margin:0; padding:0; font-family:sans-serif; font-size:11pt; color:black">
Get <a href="https://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Faka.ms%2Fghei36&data=02%7C01%7C%7Cdfc9f42b6bf74a8c212608d72b618d1d%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637025570288546926&sdata=7qeRJjXIWY%2FODCN%2BcD%2Bq3xv6l0LZQY8S8JhBvmHCbMk%3D&reserved=0" originalsrc="https://aka.ms/ghei36" shash="G3S2i3c2xfHE5IcdQO6SWMjlDRFaP7qfj+v9ejUyRxMwCa4vD6slkEaT78ABv+KuH1GzsSFPL8SZUru7ptU9Fdw1a1PsBz9x0ZtCBCEtwSzBDxBSMvYnwSr6It69URQ7FxA4LxUVPDsEa6MuQ2GkRG73OSUQBLBXYmPpE/L5L2o=">
Outlook for Android</a></div>
</span><br>
</div>
<hr tabindex="-1" style="display:inline-block; width:98%">
<div id="x_x_divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" color="#000000" style="font-size:11pt"><b>From:</b> chirp_devel-bounces@intrepid.danplanet.com <chirp_devel-bounces@intrepid.danplanet.com> on behalf of Derek Chauran via chirp_devel <chirp_devel@intrepid.danplanet.com><br>
<b>Sent:</b> Tuesday, August 27, 2019 8:56:24 PM<br>
<b>To:</b> chirp_devel@intrepid.danplanet.com <chirp_devel@intrepid.danplanet.com><br>
<b>Subject:</b> [chirp_devel] [PATCH] Fixing issue where KGUV8D Plus did not allow savinng memories with certain valid tuning steps</font>
<div> </div>
</div>
</div>
<font size="2"><span style="font-size:11pt">
<div class="x_PlainText"># HG changeset patch<br>
# User Derek Chauran <af7ux@outlook.com><br>
# Date 1566956276 25200<br>
# Tue Aug 27 18:37:56 2019 -0700<br>
# Node ID d5665709c5ec7c0549645d75b955592ab826cc74<br>
# Parent f2f71d0cd6a9a2eadd18b6315afd6078a6519632<br>
Fixing issue where KGUV8D Plus did not allow savinng memories with certain valid tuning steps.<br>
<br>
diff --git a/chirp/drivers/kguv8dplus.py b/chirp/drivers/kguv8dplus.py<br>
--- a/chirp/drivers/kguv8dplus.py<br>
+++ b/chirp/drivers/kguv8dplus.py<br>
@@ -519,6 +519,7 @@<br>
rf.valid_bands = [(134000000, 175000000), # supports 2m<br>
(400000000, 520000000)] # supports 70cm<br>
rf.valid_characters = chirp_common.CHARSET_ASCII<br>
+ rf.valid_tuning_steps = STEPS<br>
rf.memory_bounds = (1, 999) # 999 memories<br>
return rf<br>
<br>
_______________________________________________<br>
chirp_devel mailing list<br>
chirp_devel@intrepid.danplanet.com<br>
<a href="https://eur04.safelinks.protection.outlook.com/?url=http%3A%2F%2Fintrepid.danplanet.com%2Fmailman%2Flistinfo%2Fchirp_devel&data=02%7C01%7C%7Cdfc9f42b6bf74a8c212608d72b618d1d%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637025570288546926&sdata=SabxgHlkuAEGqXOeRfxwoF%2FCyBox%2FIo6dXLuQhCri38%3D&reserved=0" originalsrc="http://intrepid.danplanet.com/mailman/listinfo/chirp_devel" shash="Ysf0iR7lrvQwq/givBhMIDerbSj/zEr80cnDcySksYS01P2WiLfj/H1NjxLMNF+8p8hCBZ9E/wxb/wxiZ/d8iFaPjSOlJpF8Kk4zt8bUJgx8lSXOnjdpyeBICeZjboo0FLPsgobvcMP+4XEVcZNxW870kEmoILw50eG8LWbAA90=">http://intrepid.danplanet.com/mailman/listinfo/chirp_devel</a><br>
Developer docs: <a href="https://eur04.safelinks.protection.outlook.com/?url=http%3A%2F%2Fchirp.danplanet.com%2Fprojects%2Fchirp%2Fwiki%2FDevelopers&data=02%7C01%7C%7Cdfc9f42b6bf74a8c212608d72b618d1d%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637025570288556939&sdata=kTPt7RXqzLKXM%2BsxK7cTJnHEs9el3Cw2v9RVLVr0qQQ%3D&reserved=0" originalsrc="http://chirp.danplanet.com/projects/chirp/wiki/Developers" shash="PCh1UeIzKBfYGGrZh6iiu2vTVzCi30+qF6HMpYusocsV7m6Wl+gSZPJtUHj0iGRcxNebAlSNXstcbIaBLoUegRD7FVeNvDs3f8fhhw5jRxDygL0dTdCrOaJU9KrAxcYiUaxY51q7hXPgpMtOODK2u/4eZFYh7jDqh/6ZOdAHt2I=">
http://chirp.danplanet.com/projects/chirp/wiki/Developers</a><br>
</div>
</span></font></div>
</div>
</body>
</html>