On Tue, Jan 05, 2010 at 10:46:34AM +0100, Sebastien Termeau wrote: > On Mon, Jan 4, 2010 at 10:03 AM, Carlo Marcelo Arenas Belon < > care...@sajinet.com.pe> wrote: > > > On Tue, Dec 29, 2009 at 02:49:28PM +0100, Sebastien Termeau wrote: > > > > > > OK, I will provide you with two new patches that include those remarks. > > > > BUG249 (the one about using tables for formatting of the host view) is IMHO > > already closed, and unless you really meant to (as I expected and asked > > before > > but got no confirmation) to be really an enhancement that would be released > > with some 3.1 version (most likely 3.1.7). > > > > if that is the case, please update the target on the bugs or if you can't > > do > > that let me know and I would do so and track the corresponding backport for > > the release. > > I agree. > How do I change the target version? It is the version number in the bug > description?
no it is the "Version" field in the details section, which now says "Trunk" and should say instead "3.1.x"; if you have problems changing that let me know and I'll do the honors. assume 3.1.7 would be OK since we are almost "code freeze" for 3.1.6, and then will prepare a backport patch that could be pulled manually and if you don't file one yourself too and which you are probably using already anyway for your local package. > > BUG250 will need an updated patch that can be applied cleanly to trunk so > > that it can be tested/enhanced further. > > I just submitted a new version of the patch. > This one can be cleanly applied to trunk. > I slightly modified the order in which thinks are done in graph.php in order > to calculate the 'start' and 'end' values before calling the metric.php > script. cool, will check it and commit it to trunk then if it is working, but I suspect someone with a better clue about UI design might have a word about it that I have before it can get into 3.1 > > > I was also thinking of adding a third one with minimum, maximum and > > average. > > > Do you think it might be interesting to have this graph also? > > > > AFAIK, those values are already in the metric graphs as numeric values, and > > the MAX is also graphed with a red line, is that what you were looking to > > add? > > Yes it is. > I was thinking that maybe the normal graphs should not come with this max > line. OK, I think this was done before, where the red line was actually a per cluster max or the real max (like 100% in a percentage metric) there were some patches also flying around to add those values to the Y axis which I am not sure got committed but which will be complimentary to your idea. > And instead, we can provide a new 'trend graph' with MIN, MAX and AVG drawn > as lines. in that case it would probably make more sense to have a checkbox in the bar to toggle trending ON/OFF so that all graphs in the host view will be showing either the normal or the trending graph, instead of having a link for each graph. Carlo ------------------------------------------------------------------------------ This SF.Net email is sponsored by the Verizon Developer Community Take advantage of Verizon's best-in-class app development support A streamlined, 14 day to market process makes app distribution fast and easy Join now and get one step closer to millions of Verizon customers http://p.sf.net/sfu/verizon-dev2dev _______________________________________________ Ganglia-developers mailing list Ganglia-developers@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/ganglia-developers
