Welcome to the Cumulus Support forum.

Latest Cumulus MX release 3.11.4 (build 3133) - 25 May 2021 (please see announcement regarding releases since 3.5.0)
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

Highstock Temperature chart - after Beteljuice script mod

Share your Cumulus MX user interface here

Moderator: mcrossley

Post Reply
Vegit8
Posts: 103
Joined: Fri 27 Sep 2013 2:11 pm
Weather Station: Davis Vantage Pro2
Operating System: Win 10
Location: Dorset
Contact:

Highstock Temperature chart - after Beteljuice script mod

Post by Vegit8 »

Hi (again)

I applied the script that was suggested in my other question (viewtopic.php?f=28&t=17825) - which seemed to go well.
Except for an oddity with the 'indoor temp' values. These seem to be picking up another value.

Transposed values are as follows..
Apparent is showing values for Dew Point
Inside is showing values for Apparent


I followed the instructions in Beteljuice's script to the letter.
Anyone else have a similar problem.

Standard TrendsT file Image Modified TrendsT Image

Cheers

User avatar
beteljuice
Posts: 3210
Joined: Tue 09 Dec 2008 1:37 pm
Weather Station: None !
Operating System: W10 - Threadripper 16core, etc
Location: Dudley, West Midlands, UK

Re: Highstock Temperature chart - after Beteljuice script mod

Post by beteljuice »

Please provide a link to your site and / or page.

There are times when we get close to psychic, but it is hard work :o
Image
......................Imagine, what you will KNOW tomorrow !

Vegit8
Posts: 103
Joined: Fri 27 Sep 2013 2:11 pm
Weather Station: Davis Vantage Pro2
Operating System: Win 10
Location: Dorset
Contact:

Re: Highstock Temperature chart - after Beteljuice script mod

Post by Vegit8 »


User avatar
beteljuice
Posts: 3210
Joined: Tue 09 Dec 2008 1:37 pm
Weather Station: None !
Operating System: W10 - Threadripper 16core, etc
Location: Dudley, West Midlands, UK

Re: Highstock Temperature chart - after Beteljuice script mod

Post by beteljuice »

What is this ?
lose_plot = [
/* example
["What", "something else"],
*/
["Temperature", "Dew Point", "Wind Chill", "Heat Index"]
<!--["Temperature", "Apparent"]-->
];
You can't use html comment in the middle of a js object !
Image
......................Imagine, what you will KNOW tomorrow !

Vegit8
Posts: 103
Joined: Fri 27 Sep 2013 2:11 pm
Weather Station: Davis Vantage Pro2
Operating System: Win 10
Location: Dorset
Contact:

Re: Highstock Temperature chart - after Beteljuice script mod

Post by Vegit8 »

oh... I was having problems getting the names with spaces to work, so I experimented with a single named variable. REMing out the others - didn't realise that wouldnt work.
Then flipped it the other way once I had proved just one worked.
Sorry

Vegit8
Posts: 103
Joined: Fri 27 Sep 2013 2:11 pm
Weather Station: Davis Vantage Pro2
Operating System: Win 10
Location: Dorset
Contact:

Re: Highstock Temperature chart - after Beteljuice script mod

Post by Vegit8 »

Its me again.

I have removed the errant row(s) from the template file and have checked the resulting page displayed on't net.

Original problem still exists ...

User avatar
beteljuice
Posts: 3210
Joined: Tue 09 Dec 2008 1:37 pm
Weather Station: None !
Operating System: W10 - Threadripper 16core, etc
Location: Dudley, West Midlands, UK

Re: Highstock Temperature chart - after Beteljuice script mod

Post by beteljuice »

Oh dear .....

The beteljuice script seems to be getting confused with the sequence of things :oops:

I don't know if I can fix it as a simple 'patch' file or not :cry:

... to be continued ...
Image
......................Imagine, what you will KNOW tomorrow !

User avatar
beteljuice
Posts: 3210
Joined: Tue 09 Dec 2008 1:37 pm
Weather Station: None !
Operating System: W10 - Threadripper 16core, etc
Location: Dudley, West Midlands, UK

Re: Highstock Temperature chart - after Beteljuice script mod

Post by beteljuice »

Ah ....

I think I've found the problem .... ajax is loading required (series) files in a dedicated order when I am removing parts of that order.

ATM I can't see how I can compensate without (small) mods to cumuluscharts.js, which defeats the purpose of a non-intrusive 'plug-in' :bash:

I'll have a think about it .....
Image
......................Imagine, what you will KNOW tomorrow !

User avatar
beteljuice
Posts: 3210
Joined: Tue 09 Dec 2008 1:37 pm
Weather Station: None !
Operating System: W10 - Threadripper 16core, etc
Location: Dudley, West Midlands, UK

Re: Highstock Temperature chart - after Beteljuice script mod

Post by beteljuice »

By George I think I've beat the beggar ......

Please change:

Code: Select all

// only used for initial graph load
let observer = new MutationObserver(mutationRecords => {
	if(chart){ 
		observer.disconnect();
		rollout();
	}
});
To:

Code: Select all

// only used for initial graph load
let observer = new MutationObserver(mutationRecords => {
	if(chart.loadingShown === false){ 
		observer.disconnect();
		rollout();
	}
});
... and tell me if it works or not !!!
Image
......................Imagine, what you will KNOW tomorrow !

Vegit8
Posts: 103
Joined: Fri 27 Sep 2013 2:11 pm
Weather Station: Davis Vantage Pro2
Operating System: Win 10
Location: Dorset
Contact:

Re: Highstock Temperature chart - after Beteljuice script mod

Post by Vegit8 »

Yay....!!! :clap:

Perfecto

Thank you for your efforts

G

User avatar
beteljuice
Posts: 3210
Joined: Tue 09 Dec 2008 1:37 pm
Weather Station: None !
Operating System: W10 - Threadripper 16core, etc
Location: Dudley, West Midlands, UK

Re: Highstock Temperature chart - after Beteljuice script mod

Post by beteljuice »

IMPORTANT BUG FIX 27 Feb 2020

As above, serious bug discovered and fixed.

Anybody using this should change to the new code
posted viewtopic.php?f=14&t=17684&p=138172#p136911
Image
......................Imagine, what you will KNOW tomorrow !

Post Reply