another little bug is that when flag is set to ‘0’ in conf file, it is treated
as ‘true’ in forecast_compact.inc.
fix is to convert the flags to int (they are strings when derived from conf
file) e.g. around line 60
#set $show_pop = int($local_settings.get('show_pop', $show_pop))
(while this extension was not being updated, i found it easier to hack from a
distance and wrap it in iframe. now that it is being updated, i need to
integrate it properly so am interested in bug fixes…)
> On 17 Aug 2020, at 7:21 pm, Graham Eddy <[email protected]> wrote:
>
> many thanks for resurrecting this extension to python3
>
> i don't use WU but a bug seems to have carried over from previous version
> (b5): forecast_compact.inc refers to forecast_iconic e.g.
> $varExists('Extras.forecast_iconic_settings')
> but is documented as using label forecast_compact e.g.
> [forecast_compact_settings]
> the nett result being incorrect config header required: [Extras]
> [[forecast_iconic_settings]]
>
> fix needs simple substitutions in half-dozen lines at top and one line at
> bottom to forecast_compact.inc
--
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/26F9C1B3-DD10-4F9D-9395-D1769357D794%40gmail.com.
