[Rivet] RatioPlotYSize or RatioPlotSizeY?

[Jonathan Butterworth]

Hi all,

There seems to be some confusion in make-plots between these two 
attributes. The documentation says RatioPlotYSize but only 
RatioPlotSizeY actually works for me. Looking at the code, both appear, 
in a way which looks buggy to me (see below), but which is confusing 
enough that I am not 100% sure.

I changed the documentation to say RatioPlotSizeY since that works for 
me, but having looked at the source code I hesistate to change that in 
case there are reasons I haven't appreciated...

Should I change it all to RatioPlotSizeY ?

Cheers,
Jon


self.description['RatioPlotSizeY'] = 0.
         if self.description.has_key('MainPlot') and 
self.description['MainPlot']=='0':
             self.description['RatioPlot'] = '1'
             self.description['PlotSizeY'] = 0.
         if self.description.has_key('RatioPlot') and 
self.description['RatioPlot']=='1':
             if self.description.has_key('RatioPlotYSize') and 
self.description['RatioPlotYSize']!='':
                 self.description['RatioPlotSizeY'] = 
float(self.description['RatioPlotYSize'])
             else:
                 if self.description.has_key('MainPlot') and 
self.description['MainPlot']=='0':
                     self.description['RatioPlotSizeY'] = 6.
                 else:
                     self.description['RatioPlotSizeY'] = 3.

-- 
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Prof. Jonathan Butterworth,              http://www.hep.ucl.ac.uk/~jmb/
Head, Physics and Astronomy Department            Tel: +44 20 7679 3444
University College London                 Gower St, London WC1E 6BT, UK
ATLAS, CERN                                       Tel: +41 22 76  72340
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~