<div dir="ltr">Thank Alexandre for that detailed list. It is very helpful. I will address them in the next couple of weeks, and now that Miller has suggested a path forward for committing the changes, hopefully the fixes show up in the official releases soon.<div><br></div><div>As for Max/MSP versions, yes I have had plans for a while, and it is on my list to be done this summer, if something else does not take over my allocated time.</div><div><br></div><div>cheers,</div><div>Shahrokh</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Jun 7, 2015 at 11:45 PM, Alexandre Torres Porres <span dir="ltr"><<a href="mailto:porres@gmail.com" target="_blank">porres@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hello, great to hear from you. Yes, I got that fix, but I never compiled it and was just hoping someday it'd show up as an update, and I'm just worried it never came to be. Sorry to put you on the spot. I'm still learning about the bug reports work and get dealt with in Pd, people in the list were explaining to me today in fact. But since everyone is gathered here, instead of me making a bug report, I'll just post everything I found wrong or funny. I spent a great deal of time testing, here we go:<div><br></div><div>- the help file of [expr] gives the wrong link to expr documentation, it points to <a href="http://www.crca.ucsd.edu/~yadegari/expr.html" target="_blank">http://www.crca.ucsd.edu/~yadegari/expr.html</a> where it should point to the new link <a href="http://yadegari.org/expr/expr.html" target="_blank">http://yadegari.org/expr/expr.html</a></div><div><br></div><div>- The random function is not listed in the documentation link above. But it's great and I always use it. It has a bug in Pd extended 0.42 where [expr random(1000, 100001)] will eventually give negative numbers. But, somehow, this seems to be fixed in vanilla 0.46-6.</div><div><br></div><div>- the "if" function is mentioned twice in the documentation<br></div><div><br></div><div>- The function "Sum" is not working yet.</div><div><br></div><div>- In the code we see other functions for tables that are not mentioned on the </div><div>documentation link They are: "avg", "Avg" and "Store" - but none of which seem to be working as well!</div><div><br></div><div>- there's this "copysig" issue (which is written wrong int the documentation as "copysign") and it seems to not be working.</div><div><br></div><div>- expr will consider integers as integers, so [expr 5 / 4] results in "1". You can do something like [expr 5./ 4] for it to work, or use the float function like this:  [expr float(5) / 4]. Now, this may not be a bug or wrong, but if you use [expr float(1.4)] it'll return you "1" instead of "1.4", and I guess it should be "1.4" anyway...</div><div><br></div><div>- [expr imodf($f1)] and [expr ldexp($f1)] are not loading</div><div><br></div><div>- [expr drem($f1)] loads with one argument, but the documentation says it takes two arguments. Even though it loads, it doesn't seem to be working, it sends lots of errors, such as "error: expr: FUNC_EVAL(1291): bad right type 0 /// expr: bang: unrecognized result 0 /// error: expr: FUNC_EVAL(1291): bad right type 3</div><div><br></div><div>- One's comlpement will load like this [expr $f1 ~ $f2]. Not sure what it's supposed to be, but it seems the second argument doesn't affect it at all.</div><div><br></div><div>- "floor" and "ceil" will load with two arguments, documentation says it was only one! Not sure what they're supposed to be, but it doesn't seem right cause it always returns the 1st argument in both of them.</div><div><br></div><div>- "fmod", wrong number of arguments in the documentation, it takes two!</div><div><br></div><div>- In the documentation, you have "</div><table cellpadding="0" cellspacing="0" border="0" width="100%" style="font-family:Times"><tbody><tr><td valign="middle" width="70"><font face="Helvetica, Arial, sans-serif"><small>atan()</small></font><font face="Helvetica, Arial, sans-serif"><small><br></small></font></td><td valign="top" align="center" width="50"><font face="Helvetica, Arial, sans-serif"><small>1</small></font><font face="Helvetica, Arial, sans-serif"><small><br></small></font></td><td valign="middle" align="left"><font face="Helvetica, Arial, sans-serif"><small>inverse hyperbolic</small></font><font face="Helvetica, Arial, sans-serif"><small> tangent"</small></font></td></tr></tbody></table><div>but I belive it should be "atan<b style="text-decoration:underline"><i>h</i></b>"<br></div><div><br></div><div>- Code says</div><div><br></div><div><div> *      The following are done but not popular enough in math libss</div><div> *      to be included yet</div><div> *              hypoth - Euclidean distance function</div><div> *              trunc</div><div> *              round</div><div> *              nearbyint - </div><div> </div></div><div><br></div><div>So, maybe they are popular enough nowadays? Maybe they're ready to be included?</div><div><br></div><div>Well, that's what I found so far. I'm no mathematician, so I can't find anything else I guess. I could test other functions but I wouldn't know if they are correct.</div><div><br></div><div>Anyway, sorry I can't help more getting into the code issues. I'm at a point now that I wish I know more about programming so I can help Pd development.</div><div><br></div><div>Thanks</div><div><br></div><div><br></div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">2015-06-08 3:08 GMT-03:00 Shahrokh Yadegari <span dir="ltr"><<a href="mailto:sdy@ucsd.edu" target="_blank">sdy@ucsd.edu</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi Alexandre,<div><br></div><div>This seems to be a bug. I am not sure why I missed the 'n' on the name either. As it seems that this function has never worked, we should fix the name as well. The solution is simple as follows:</div><div><br></div><div>







<p>extra/expr~/vexp_fun.c</p>
<p>188c188</p>
<p><         {"copysign",    ex_copysign,    2},</p>
<p>---</p>
<p>>         {"copysig",     ex_copysign,    1},</p><p><br></p><p>I checked the mailing list (I am sorry that I am not a regular reader) and I see some concerns about the maintenance of the code. I am happy to fix the problems and I thought I sent you a fix for the last problem you had found in 2009. Can somebody point me to a document or a webpage that explains what the best way is to introduce new changes to the pd source base.  Miller, should I pass the fixes to you, or is there another repository?</p><p>In the meanwhile, if there are other bugs in expr that anybody else knows, please send them to me and I will fix them as soon as I can.</p><p>best,<br>Shahrokh</p></div></div><div class="gmail_extra"><div><div><br><div class="gmail_quote">On Sun, Jun 7, 2015 at 12:57 PM, Alexandre Torres Porres <span dir="ltr"><<a href="mailto:porres@gmail.com" target="_blank">porres@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">hi there, I was checking the source of expr (find it attached) and saw this "copysign" function that I needed for a formula, but I can't get it to work/load.<div><br></div><div>I'm trying something like [expr copysign($f1)] and it's not happening. The expr manual page in <a href="http://yadegari.org/expr/expr.html" target="_blank">http://yadegari.org/expr/expr.html</a> tells about this function as well, here it is:</div><div><br></div><div><table cellpadding="0" cellspacing="0" border="0" width="100%" style="font-family:Times"><tbody><tr><td valign="top" align="left"><small><font face="Helvetica, Arial, sans-serif">copysign()<br></font></small></td><td valign="top" align="center"><small><font face="Helvetica, Arial, sans-serif">1<small><br></small></font></small></td><td valign="top"><small><font face="Helvetica, Arial, sans-serif">copy sign of a number</font><font face="Helvetica, Arial, sans-serif">(added in version 0.4)<br></font></small></td></tr></tbody></table></div><div><br></div><div>So it seems I'm supposedly doing the right thing... is it a bug? I'm on extended 0.42 and it says that it's expr version 0.4 when it loads.</div><div><br></div><div>cheers</div></div>
</blockquote></div><br><br clear="all"><div><br></div></div></div><span><font color="#888888">-- <br><div><div dir="ltr">Shahrokh Yadegari<br>Professor of Composition and Sound Design,<br>Theatre and Dance Department<br>University of California, San Diego<br>Director, Initiative for Digital Exploration of Arts and Science, (IDEAS)<br>California Institute for Telecommunications and Information Technologies (Calit2)<br>Email: <a href="mailto:sdy@ucsd.edu" target="_blank">sdy@ucsd.edu</a><br>Web: <a href="http://yadegari.org" target="_blank">http://yadegari.org</a><br>Tel: <a href="tel:%28858%29%20822-4113" value="+18588224113" target="_blank">(858) 822-4113</a><br>Fax: <a href="tel:%28858%29%20534-1080" value="+18585341080" target="_blank">(858) 534-1080</a><br><br></div></div>
</font></span></div>
</blockquote></div><br></div>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr">Shahrokh Yadegari<br>Professor of Composition and Sound Design,<br>Theatre and Dance Department<br>University of California, San Diego<br>Director, Initiative for Digital Exploration of Arts and Science, (IDEAS)<br>California Institute for Telecommunications and Information Technologies (Calit2)<br>Email: <a href="mailto:sdy@ucsd.edu" target="_blank">sdy@ucsd.edu</a><br>Web: <a href="http://yadegari.org" target="_blank">http://yadegari.org</a><br>Tel: (858) 822-4113<br>Fax: (858) 534-1080<br><br></div></div>
</div>