Replies: 10 comments 3 replies
-
Hi Jon,
There are a few bugs in 3.2.0 you should be aware of, most to mention a bug with the date/time display on the new sensor status page (see this issue GH-172 and this commit 178e6ba) Otherwise, I am happy that you are testing it out, and I would be happy to get any feedback ;) For the tide display: I wasn't aware that tides are shown in feet, the observation Diff: 8191a4b It should work, feel free to test it out! I will have a look at why this does not get displayed in the correct unit because this should not be statically tied to a unit. |
Beta Was this translation helpful? Give feedback.
-
Hi Dave Sorry no joy! Even with 3.2.0 now installed. see wslwx.co.uk. BW Jon
|
Beta Was this translation helpful? Give feedback.
-
Hi Dave That was just the copy of the forecast-table.inc from the link above... I did not play with that one at all... Jon |
Beta Was this translation helpful? Give feedback.
-
Hi Dave Weird. OK I will download again and try it. Most strange.... Jon |
Beta Was this translation helpful? Give feedback.
-
Hi Dave These are the steps just taken to get the same result.... even weirder! Jon jon@weewx:/etc/weewx/skins/weewx-wdc/includes$ sudo wget https://github.com/Daveiano/weewx-wdc/blob/174-show-tides-in-meters/skins/weewx-wdc/includes/forecast-table.inc sudo tail -f /var/log/syslog jon@weewx:/etc/weewx/skins/weewx-wdc/includes$ sudo mv forecast-table.inc forecast-table.inc.2 sudo tail -f /var/log/syslog |
Beta Was this translation helpful? Give feedback.
-
Ah! That looks better! Now just got to change the text to read meters, that I can do! Thank you Jon |
Beta Was this translation helpful? Give feedback.
-
Hmm Perhaps I can't change that text so easily! Looks like it is calling from somewhere else. Jon |
Beta Was this translation helpful? Give feedback.
-
You can just replace Note: You need to do this twice: One time for the daily and one time for the hourly forecast. Here: https://github.com/Daveiano/weewx-wdc/blob/174-show-tides-in-meters/skins/weewx-wdc/includes/forecast-table.inc#L341 and here: https://github.com/Daveiano/weewx-wdc/blob/174-show-tides-in-meters/skins/weewx-wdc/includes/forecast-table.inc#L676 BTW: Which unit_system are you using for your reports? US or metric? Read more here: https://weewx.com/docs/customizing.htm#how_to_change_units |
Beta Was this translation helpful? Give feedback.
-
Hi Jon, I just tested around a bit with the tides and noticed that everything is correctly displayed. If you are seeing tides in feet, then it's because of your WeeWX configuration. The observation is called On your page, I can see that you are using feet for So you could either switch to meter for the whole This was just a quick summary of what I have found out |
Beta Was this translation helpful? Give feedback.
-
Hi Dave Ah yes! The great British habit of mixing units! Unfortunately Weewx itself does not cope with us Brit's and their passion for the mixed unit schema. Never mind, the hack done works until I can come up with something more friendly. It just takes a little longer to deploy a new version... Thank you for looking and your advice... Jon |
Beta Was this translation helpful? Give feedback.
-
Hi Dave
Many thanks for v3.2.0. I have only just seen it and so will get it deployed as soon as possible. Switched on notifications correctly now!
Hopefully a simple one, I would like to display the tide heights in meters. Someone else has done this in the forecast skin tide page but unfortunately it does not work in the tide-forecast include page.
Is this simple to implement?
BW
Jon
Code:
Beta Was this translation helpful? Give feedback.
All reactions