From: Antonio Ospite Date: Thu, 29 Nov 2018 10:31:24 +0000 (+0100) Subject: Fix errors with recent gnuplot versions X-Git-Url: https://git.ao2.it/experiments/gnuplot_seasonal_plot.git/commitdiff_plain/1b00c99a236bd3729b29efe9b43b20d4f42330ef?ds=sidebyside Fix errors with recent gnuplot versions It looks like recent gnuplot versions do not keep variables around when they get out of scope, like a counter used only inside a loop. This results in some errors in the modular scripts: ./modular_cartesian.gnuplot", line 62: undefined variable: i ./modular_polar.gnuplot", line 59: undefined variable: i Fix this by using 'year_max' instead of 'i' outside of the loop where 'i' was defined. Since 'year_max' is a float variable string concatenation would not work with it, resulting in the following errors: ./modular_cartesian.gnuplot", line 62: internal error : STRING operator applied to undefined or non-STRING variable ./modular_polar.gnuplot", line 59: internal error : STRING operator applied to undefined or non-STRING variable So while at it also convert the float to an integer so that the result can be used with the string concatenation operator. --- diff --git a/modular_cartesian.gnuplot b/modular_cartesian.gnuplot index 07f7aeb..574806c 100755 --- a/modular_cartesian.gnuplot +++ b/modular_cartesian.gnuplot @@ -59,7 +59,7 @@ cblabels = 'set cbtics ( ' do for [i=year_min:year_max] { cblabels = cblabels . '"' . i .'" ' . i .',' } -cblabels = cblabels . '"" ' . (i + 1) .',' +cblabels = cblabels . '"" ' . int(year_max + 1) .',' cblabels = cblabels . ' )' eval(cblabels) diff --git a/modular_polar.gnuplot b/modular_polar.gnuplot index 4c00f2f..2007101 100755 --- a/modular_polar.gnuplot +++ b/modular_polar.gnuplot @@ -56,7 +56,7 @@ cblabels = 'set cbtics ( ' do for [i=year_min:year_max] { cblabels = cblabels . '"' . i .'" ' . i .',' } -cblabels = cblabels . '"" ' . (i + 1) .',' +cblabels = cblabels . '"" ' . int(year_max + 1) .',' cblabels = cblabels . ' )' eval(cblabels)