On Thu, Sep 25, 2014 at 5:41 PM, Dan Smith <dsmith@danplanet.com> wrote:
Hi Ron,

> Attached is my driver for the Wouxun KG-UV6D.

Has anyone else (Ron, you included) looked at this code and compared it
to others to see if it's an obvious clone of something else? It didn't
appear to be obvious to me. Jim?

As far as I know it is different. It seems that Wouxun changed how they do thing with their mobile radios and the KG-UV8D.

This it the kind of help that I was looking for to get going with the KG-UV920P mobile radio (and possibly the UV950-P after that).




This should be indented to the same level as the code it pertains to.

> +        rs = RadioSetting("ponmsg", "Poweron message",
> +                         RadioSettingValueList(
> +                             PONMSG_LIST,
> +                             PONMSG_LIST[self._memobj.settings.ponmsg]))
> +     cfg_grp.append(rs)
> +        rs = RadioSetting("voice", "Voice Guide", RadioSettingValueBoolean(
> +                         self._memobj.settings.voice))
> +        cfg_grp.append(rs)
> +        rs = RadioSetting("timeout", "Timeout Timer",
> +                         RadioSettingValueInteger(15, 900,
> +                             self._memobj.settings.timeout * 15, 15))
> +        cfg_grp.append(rs)
> +     rs = RadioSetting("toalarm", "Timeout Alarm",
> +                         RadioSettingValueInteger(0, 10,
> +                             self._memobj.settings.toalarm))
> +     cfg_grp.append(rs)
> +     rs = RadioSetting("channel_menu", "Menu available in channel mode",
> +                         RadioSettingValueBoolean(
> +                             self._memobj.settings.channel_menu))
> +     cfg_grp.append(rs)
> +        rs = RadioSetting("power_save", "Power save",
> +                         RadioSettingValueBoolean(
> +                             self._memobj.settings.power_save))
> +        cfg_grp.append(rs)

Looks like some alignment issues here. Just to be clear, each indent
level should be four spaces.

Yeah. I have to convert tabs to spaces with my editor and everything was indented as it should be. There are a few links longer than 80 characters that should be addressed too.

Again, thanks a lot. This looks really close and I'll be glad to get
this into the tree soon!
 
I was so hoping to get this going to at least attempt a download with my KG-UV920P. Unfortunately, I get this error.

Traceback (most recent call last):
  File "C:\Users\Root\chirp.hg\chirpui\mainapp.py", line 1391, in mh
    self.do_download(*args)
  File "C:\Users\Root\chirp.hg\chirpui\mainapp.py", line 641, in do_download
    not self._confirm_experimental(rclass):
  File "C:\Users\Root\chirp.hg\chirpui\mainapp.py", line 599, in _confirm_experi
mental
    can_squelch=True)
  File "C:\Users\Root\chirp.hg\chirpui\common.py", line 350, in show_warning
    textview = _add_text(d, text)
  File "C:\Users\Root\chirp.hg\chirpui\common.py", line 310, in _add_text
    v.get_buffer().set_text(text)
TypeError: GtkTextBuffer.set_text() argument 1 must be string or read-only buffe
r, not None

I couldn't find the debug.log file. Doesn't the developer version write the debug.log to the same place as a regular install of CHIRP? Do I not have something enabled?

Jim