Hi Alexandre,
Awesome, can't wait for the new release :)
The source I am using is some 0.44 version.
You mean the 0.44.0 extended, right?
Sorry, yes pd-extended 0.44. If all these objects were in vanilla, our world would look quite different :-)
Well, I guess the next step is to see if we can get a new Pd Extended soon, and hopefully in pace with the current 0.46 version.
But this is another discussion.
By making these kinds of changes and improvements, we can make a new release more worthwhile, and thus increase the change it will happen ;-)
Thanks
Greetings,
Fred Jan
2014-12-13 16:53 GMT-02:00 Fred Jan Kraan <fjkraan@xs4all.nl mailto:fjkraan@xs4all.nl>:
Hi Alexandre, > Hi, I re-sending (or forwarding) my first message about cartopol, with > the patch I had attached to exemplify the issue. My message was brief > then, but it said it all. > The message was clear, surely with the additional info. Attached is an image of your patch with a modified poltocar~. > > Now, I'm assuming that the version in 0.43 is the latest and is the one > you have. About the bug list then, the right thing to do would be to > take [cartopol~] out of the list and just keep [poltocar~], saying > something like "wrong conversion". The source I am using is some 0.44 version. The final code patch should be based on the svn code. Not much difference is to be expected here. > > I haven't checked the code. But the formulas are pretty simple, just > basically sine and cosine, so you can't go wrong by fixing it. There > must be a minus sing somewhere where it didn't belong, just take it out. Indeed it is (poltocar.c:35): *out2++ = -am * sinf(ph); /* CHECKED */ Removing the minus and recompiling made your patch work as expected. Added some info to the sourceforge bug "#1128 still on cartopol~ / poltocar~". > > Cheers Greetings, Fred Jan > > ---------- Forwarded message ---------- > From: *Alexandre Torres Porres* <porres@gmail.com <mailto:porres@gmail.com> <mailto:porres@gmail.com <mailto:porres@gmail.com>>> > Date: 2014-12-08 20:37 GMT-02:00 > Subject: Re: [PD] Cyclone suite initiative > To: Fred Jan Kraan <fjkraan@xs4all.nl <mailto:fjkraan@xs4all.nl> <mailto:fjkraan@xs4all.nl <mailto:fjkraan@xs4all.nl>>> > Cc: pd-list@mail.iem.at <mailto:pd-list@mail.iem.at> <mailto:pd-list@mail.iem.at <mailto:pd-list@mail.iem.at>> > > cartopol~ and poltocar~ are also buggy > > I pointed it out once, but only cartopol~ was corrected and it made it > worse. check atatchment. > > do you have this covered? > > cheers > > 2014-12-08 14:30 GMT-02:00 Fred Jan Kraan <fjkraan@xs4all.nl <mailto:fjkraan@xs4all.nl> > <mailto:fjkraan@xs4all.nl <mailto:fjkraan@xs4all.nl>>>: > > Hi Alexandre, > > Thanks for the report. > > Next to working on the help-patches, I am working on creating such a > list. I'll put it somewhere public when the patches are done. > > So when you find any others, I would like to know. > > Greetings, > > Fred Jan > > > hi, you probably know about this bug, but [rand~]'s argument doesn't > > really work. I'd be happy to help find other bugs, do you have a > list of > > bugs you're working on? > > > > cheers > > > >