Hi Gary,
Thanks for looking into this! Yeah, my "fix" was just me typing away in
the message, so I indeed did miss the extra ":%d". My coding may be bad,
but it ain't *that* bad! :-)
I had, up to now, the ip address for the GW1000 set to auto. I have an
atrociously difficult to manage Spectrum router (which I should replace
with a better one) that doesn't have a web interface to manage it, but
rather an app. IP allocation on that app is difficult to find, but I did
find it, and now I have changed weewx.conf to have a fixed ip address for
the GW1000. Will that change anything if weewx again loses contact with
the GW1000?
Thanks,
Bob
On Tuesday, March 15, 2022 at 11:27:27 PM UTC-7 gjr80 wrote:
> Thanks for spotting that, device_list is a list of dicts so line 3928 was
> never going to work. I am surprised it has not shown up previously, though
> it is buried in some code that is seldom used. You almost have the fix
> correct, but '%s' should be '%s:%d' to give the *IP address:port number*
> format. There are also a couple of other changes further down from line
> 3298 that will also need to be made. I should have an update in the next
> few days; the rediscovery code is complex and I want to make sure sure it
> is functioning properly (now) - testing of the rediscovery code is time
> consuming.
>
> In the meantime, are you operating your device with a fixed IP allocation?
> If possible that is the recommended way to use the GW1x00 devices with the
> GW1000 driver.
>
> Gary
>
> On Wednesday, 16 March 2022 at 04:42:02 UTC+10 Dr__Bob wrote:
>
>> Hi. From time to time, it seems like my GW1000 disappears. It looks
>> like the code is trying to rediscover it, but in the process hits a bug in
>> the code and crashes. This is with weewx 4.5.1 with python 3.7.3 and
>> GW1000 0.41. The syslog looks like:
>>
>> Mar 15 04:14:45 raspberrypi weewx[1374] ERROR user.gw1000: Failed to
>> obtain response to command 'CMD_READ_SENSOR_ID_NEW' after 3 attempts
>>
>> Mar 15 04:14:45 raspberrypi weewx[1374] INFO user.gw1000: Attempting to
>> re-discover GW1000...
>>
>> Mar 15 04:15:05 raspberrypi weewx[1374] CRITICAL user.gw1000: ****
>> Traceback (most recent call last):
>>
>> Mar 15 04:15:05 raspberrypi weewx[1374] CRITICAL user.gw1000: ****
>> File "/usr/share/weewx/user/gw1000.py", line 3587, in get_sensor_id
>>
>> Mar 15 04:15:05 raspberrypi weewx[1374] CRITICAL user.gw1000: ****
>> return self.send_cmd_with_retries('CMD_READ_SENSOR_ID_NEW')
>>
>> Mar 15 04:15:05 raspberrypi weewx[1374] CRITICAL user.gw1000: ****
>> File "/usr/share/weewx/user/gw1000.py", line 3740, in send_cmd_with_retries
>>
>> Mar 15 04:15:05 raspberrypi weewx[1374] CRITICAL user.gw1000: ****
>> raise GW1000IOError(_msg)
>>
>> Mar 15 04:15:05 raspberrypi weewx[1374] CRITICAL user.gw1000: ****
>> user.gw1000.GW1000IOError: Failed to obtain response to command
>> 'CMD_READ_SENSOR_ID_NEW' after 3 attempts
>>
>> Mar 15 04:15:05 raspberrypi weewx[1374] CRITICAL user.gw1000: ****
>>
>> Mar 15 04:15:05 raspberrypi weewx[1374] CRITICAL user.gw1000: ****
>> During handling of the above exception, another exception occurred:
>>
>> Mar 15 04:15:05 raspberrypi weewx[1374] CRITICAL user.gw1000: ****
>>
>> Mar 15 04:15:05 raspberrypi weewx[1374] CRITICAL user.gw1000: ****
>> Traceback (most recent call last):
>>
>> Mar 15 04:15:05 raspberrypi weewx[1374] CRITICAL user.gw1000: ****
>> File "/usr/share/weewx/user/gw1000.py", line 3024, in run
>>
>> Mar 15 04:15:05 raspberrypi weewx[1374] CRITICAL user.gw1000: ****
>> self.client.collect_sensor_data()
>>
>> Mar 15 04:15:05 raspberrypi weewx[1374] CRITICAL user.gw1000: ****
>> File "/usr/share/weewx/user/gw1000.py", line 2476, in collect_sensor_data
>>
>> Mar 15 04:15:05 raspberrypi weewx[1374] CRITICAL user.gw1000: ****
>> queue_data = self.get_live_sensor_data()
>>
>> Mar 15 04:15:05 raspberrypi weewx[1374] CRITICAL user.gw1000: ****
>> File "/usr/share/weewx/user/gw1000.py", line 2525, in get_live_sensor_data
>>
>> Mar 15 04:15:05 raspberrypi weewx[1374] CRITICAL user.gw1000: ****
>> self.update_sensor_id_data()
>>
>> Mar 15 04:15:05 raspberrypi weewx[1374] CRITICAL user.gw1000: ****
>> File "/usr/share/weewx/user/gw1000.py", line 2538, in update_sensor_id_data
>>
>> Mar 15 04:15:05 raspberrypi weewx[1374] CRITICAL user.gw1000: ****
>> sensor_id_data = self.station.get_sensor_id()
>>
>> Mar 15 04:15:05 raspberrypi weewx[1374] CRITICAL user.gw1000: ****
>> File "/usr/share/weewx/user/gw1000.py", line 3591, in get_sensor_id
>>
>> Mar 15 04:15:05 raspberrypi weewx[1374] CRITICAL user.gw1000: ****
>> if not self.rediscover():
>>
>> Mar 15 04:15:05 raspberrypi weewx[1374] CRITICAL user.gw1000: ****
>> File "/usr/share/weewx/user/gw1000.py", line 3928, in rediscover
>>
>> Mar 15 04:15:05 raspberrypi weewx[1374] CRITICAL user.gw1000: ****
>> gw1000_str = ', '.join([':'.join(['%s:%d' % b]) for b in device_list])
>>
>> Mar 15 04:15:05 raspberrypi weewx[1374] CRITICAL user.gw1000: ****
>> File "/usr/share/weewx/user/gw1000.py", line 3928, in <listcomp>
>>
>> Mar 15 04:15:05 raspberrypi weewx[1374] CRITICAL user.gw1000: ****
>> gw1000_str = ', '.join([':'.join(['%s:%d' % b]) for b in device_list])
>>
>> Mar 15 04:15:05 raspberrypi weewx[1374] CRITICAL user.gw1000: ****
>> TypeError: not enough arguments for format string
>>
>> I *think* the bug is on line 3928 in rediscover(self):
>>
>> gw1000_str = ', '.join([':'.join(['%s:%d' % b]) for b in device_list])
>>
>> This looks like a copy of the similar line further up in the code where
>> the GW1000 is first discovered. I'm a complete python newbie, but it looks
>> like the second join in the above line wants two fields but is only getting
>> one. Shouldn't it be ".join(['%s' % (b['ip_address'],b['port'])])"?
>> That's the way the first version of gw1000_str is formed.
>>
>> Cheers,
>>
>> Bob Clare
>>
>
--
You received this message because you are subscribed to the Google Groups
"weewx-user" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To view this discussion on the web visit
https://groups.google.com/d/msgid/weewx-user/697c7749-2be8-49eb-bfec-a2e5d9702905n%40googlegroups.com.
