Welcome to the Cumulus Support forum.

Latest Cumulus MX V3 release 3.28.6 (build 3283) - 21 March 2024

Cumulus MX V4 beta test release 4.0.0 (build 4018) - 28 March 2024

Legacy Cumulus 1 release v1.9.4 (build 1099) - 28 November 2014 (a patch is available for 1.9.4 build 1099 that extends the date range of drop-down menus to 2030)

Download the Software (Cumulus MX / Cumulus 1 and other related items) from the Wiki

BUG: Line 5067 in Cumulus.cs

From build 3044 the development baton passed to Mark Crossley. Mark has been responsible for all the Builds since. He has made the code available on GitHub. It is Mark's hope that others will join in this development, but at the very least he welcomes your ideas for future developments (see Cumulus MX Development suggestions).

Moderator: mcrossley

Post Reply
sfws
Posts: 1183
Joined: Fri 27 Jul 2012 11:29 am
Weather Station: Chas O, Maplin N96FY, N25FR
Operating System: rPi 3B+ with Buster (full)

BUG: Line 5067 in Cumulus.cs

Post by sfws »

This is not a serious bug, but a typo introduced when you presumably tried to tidy some of the code.

The code in Cumulus.cs reads from Cumulus.ini a parameter with attribute name LowTempAlarmSoundFile into a internal variable. MX used to write back that internal variable into the configuration file with the same attribute name.

Sometime (between release 3.8.2 and 3.10.4, but I am not wading through all your past commits to work out the exact build involved), a code rewrite in Cumulus.cs has introduced a bug (in the current release this in line 5067) as now MX is writing the internal variable back into Cumulus.ini using an attribute called LowTempAlarm.SoundFile i.e. a full stop as in the internal variable name has been introduced into the attribute name.

Consequently, If somebody decides to tidy up their configuration file, by deleting the file while MX is running and do a dummy "Save" on one of the settings pages to recreate the file, MX will lose any value associated with this setting!
User avatar
mcrossley
Posts: 12695
Joined: Thu 07 Jan 2010 9:44 pm
Weather Station: Davis VP2/WLL
Operating System: Bullseye Lite rPi
Location: Wilmslow, Cheshire, UK
Contact:

Re: BUG: Line 5067 in Cumulus.cs

Post by mcrossley »

Ta, fixed.
Post Reply