<br><br><div><span class="gmail_quote">2007/1/24, Thomas Friedrichsmeier <<a href="mailto:thomas.friedrichsmeier@ruhr-uni-bochum.de" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">thomas.friedrichsmeier@ruhr-uni-bochum.de

</a>>:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

Hi! On Wednesday 24 January 2007 16:35, I. Soumpasis wrote: > I will uploaded at svn in some minutes. Also changed the plot.pluginmap > accordingly.What else should I change so the plugin to be installed?

> Makefile or something? Yes, <a href="http://Makefile.am" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">Makefile.am</a>. Stefan has taken care of it already. > I made enough testings but please take a look and give advices and ideas.

<br><br>Some comments:

<br>- Try to avoid patterns like<br><br>if (<? getRK ("something")?>) {<br>        do.something.r ()<br>}<br><br>Rather write<br><br><? if (getRK_val ("something") == "TRUE") { ?>

<br>do.something.r ()<br><? } ?></blockquote><div><br>I tried something like this (for many hours) but in the code I get only the first part of the code like the first if condition is always true. I send the barplot2.php

 to take a look if I have done something wrong.<br></div><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">This way the generated code will not get convoluted with statements which are

<br>not acutally needed. See also<br><a href="http://rkward.sourceforge.net/documents/devel/plugins/phptemplate.html#policysimplicity" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">http://rkward.sourceforge.net/documents/devel/plugins/phptemplate.html#policysimplicity

</a> .<br><br>Something like (not tested for correctness):<br><br><?<br>if (getRK_val ("beside") == "TRUE") {<br>?>rk.temp.barplot <- <?<br>}<br>?>barplot((rk.temp.x)<? if (getRK_val ("rainbow")) { ?>,

<br>col=rainbow(rk.temp.x) } ?>, beside=<? getRK ("beside"); ?>, legend.text=<?<br>getRK ("legend"); ?>,  ylim = range(rk.temp.x) * c(0, 1.2))<br><?<br><br>if ((getRK_val ("beside") == "TRUE") && getRK_val ("labels")) { ?>

<br>text(rk.temp.barplot, rk.temp.x, labels=rk.temp.x, pos=<? getRK ("place"); ?>,<br>offset=.5)<br><? } ?></blockquote><div><br>That would be very nice and tried it in different ways and in this way too, but with no result. The if (if getRK_val ("rainbow")) { ?>, col=rainbow(

rk.temp.x) <? } ?>, does not seem to work for me, because this part does not appear in the code of the plug in, or appears constantly, whatever the choice of the user.  Tried the code as is with a small differentiation here l=rainbow(

rk.temp.x) <? } ?> is in the barplot4.php.<br></div><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

- You might want to rewrite the first part as:<br>rk.temp.x <- (<? echo ($vars); ?>)<br>if(is.factor(rk.temp.x)) rk.temp.x=summary(rk.temp.x)<br>else if (!is.matrix(rk.rempt.x)) error ("Only matrices or factors are

<br>supported")<br><br>Saving one variable name, and also giving a more meaningful error message in<br>case something wrong was selected.</blockquote><div><br>I tried this, but I get a syntax error. The warnings points me to the .xml file, so I removed the number type=numeric but with no result. 

<br></div><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">- in the cleanup function you could write to clear all symbols starting<br>with "

rk.temp" without listing each:<br>rm (list=grep ("^rk.temp", ls (), value=TRUE))</blockquote><div><br>This is very helpful. <br></div><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

- in the .xml, you might want to add the following to the logic section:<br><convert id="labelsvalue2" mode="and" sources="labelsvalue;labels.enabled" /><br>and then change the next line to

<br><connect client="place.enabled" governor="labelsvalue2" /></blockquote><div><br>Thanks very much for this.  I will commit it to svn.</div><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

Otherwise, if the user first chooses to display labels, and after that sets the<br>mode to stacked, the placement option will still remain enabled.<br><br>Regards<br>Thomas</blockquote><div><br>"I don't think it's necessary in the help pages to state which settings are the default." 

<br><br>I agree with this. Already removed the default references from .rkh page for this plugin and I will do so for the others too. I will commit it to svn.<br><br>Finally, I made a change making one more variable 

rk.temp.barplot2 and moved all the function to the function preprocess() as I had done in correlation graph.  This leads me to one idea that it cross my mind. Could we write some functions and embed them in the rkward package for R, so to use them in the plugins like the Rcmdr package does?

<br><br>Regards,<br>Ilias <br></div><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">-------------------------------------------------------------------------

<br>Take Surveys. Earn Cash. Influence the Future of IT

<br>Join SourceForge.net's Techsay panel and you'll get the chance to share your<br>opinions on IT & business topics through brief surveys - and earn cash<br><a href="http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">

http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV</a><br><br>_______________________________________________<br>RKWard-devel mailing list<br><a href="mailto:RKWard-devel@lists.sourceforge.net" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">

RKWard-devel@lists.sourceforge.net</a><br><a href="https://lists.sourceforge.net/lists/listinfo/rkward-devel" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">https://lists.sourceforge.net/lists/listinfo/rkward-devel

</a><br><br><br><br></blockquote></div><br>