-
-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[BUG] Reason: '<' not supported between instances of 'NoneType' and 'float' in the skins/weewx-wdc/year-%Y.html.tmpl file #256
Comments
Hello,
Henry |
I can't spot any issue and I can't even reproduce this error. Did you guys make any changes to the provided
@cbirkenbeul What issue do you mean? The template which causes the error Please try to disable the new added Stat tables via skin.conf:
This should stop the error and generate the templates. |
Hey, With disabling the stat_table fields all reports are now generated fine. |
Ok but something is going on here. Do you have any missing data? Like you gathered data since 2020 but your database does not contain values for August 2021 or something? Cant make any sense out of this Error message. Would it be possible to share your weewx db with me? This way I could debug into this with the actual data causing this. |
We had an hardware issue since then. So no software problem. I could, but I use a mariadb as backend and the sql file seems quite big. But should missing old data cause this issue? I mean, I run the 3.3 version quite long without any issues. |
The error is triggered by the One thing you could try in the ...
[[stat_tables]]
[[[tables_outtemp]]]
observation = "outTemp"
label = "Temperature Stat Table"
aggregate_types = "min", "avg", "max"
#[[[tables_rain]]]
# observation = "rain"
# label = "Rain Stat Table"
# aggregate_types = "sum", "avg"
...
Does this also solve the problem? @hoetzgit Do you have a sqlite file of your DB to reproduce the problem for me? |
Ok, I think I found the problem. Attached is an updated file Its at You can then enable back the |
I have no more errors in the log with the customized stat-table.inc |
For me it does not work. I've implemented your file and reenabled the stats_table section which I comment out some days ago. |
Ok, I don't think the changed file is related to the error. The error appears on every page, except the index page (at least it's not included in the log) - the changed file only impacts the year and statistics pages, not eg. the week or month pages which also fail to generate. There seems to be something messed up. Did you only change the one file or did you change other things, too? Perhaps copied to the wrong location? Changed something in skin.conf? |
As you told me in your very first answer, I disabled the stats_table completely. For your patch I reenabled them. I have changed the language and added the lightning values as discussed here. I've download your file with wget on my system. Renamed it and copied the file to the includes directory. I don't see any formatting errors like utf-8 or so. My procedure should be fine. |
Sorry for not getting back to you sooner, I got a nasty cold last week... It's really difficult for me to reproduce these error messages. The first thing would be to disable the stat-tables again, I guess. This should solve the errors. The error from your logs So if you want to share your DB with me I would have a look to reproduce the issue (you can share it with [email protected]). In the meantime, disabling the stat-tables again should work. |
Just upgraded to weewx 5.0.2 & weewx-wdc 3.5.0. Got the same error but on a different template. Change to |
@Daveiano Its still to big to send by mail. I've uploaded it to google drive. Please let me now If you downloaded the file, so I can delete it. Thanks. |
Ok, I downloaded the file. Will set up a testing environment and have a look in the next days. |
Seems like I am having similiar issues.
|
Yes looks like you got two problems here. The
Looks like weewx has problems to calculate the For the @donadroni Out of curiosity: Do you use a MySQL or a SQLite DB with weewx? |
I am using the default so it is SQLite |
@cbirkenbeul I am really out of ideas now. I just set up a local MySQL server and imported the provided DB. I can run the report without any errors, everything gets generated. If I remove the change from b56b5fc, the error re-appears again. Here is a screenshot of the stat-tables, which trigger the error, no problems at all: Just to be sure, I released v3.5.1 which includes the fix. @cbirkenbeul @donadroni Could you please test with v3.5.1? |
I just freshly installed weewx from and installed wdc 3.5.1 with my database but default config.
-- edited |
Yeah, looks to me like something is going on with your station. Good luck with checking! But: the original |
Good news from my site. Now update the weewx.conf and skin.conf with my customization (language, adding observations) and restarted weewx again. Still no error message. I will have a look tomorrow if it still dont throw any errors. Thanks for your help! |
@cbirkenbeul @donadroni Since there was no response to the issue for about a week, I consider this closed. Wil keep the issue open until the end of the week though. In case there is some feedback on the way ;) |
Hello,
I've upgraded to weewx 5.0.2 and your skin to 3.5 and got a lot of issues. Most I could fix by myself because of the whew 5 upgrade.
But this issue I got and cannot solve by myself:
There were a fixed issue with the same template file that was solved for 3.5, but I got a different error. The message seems that it could be a skin issue. Otherwise I would ask in the google groups.
Thanks.
The text was updated successfully, but these errors were encountered: