Patches item #3603652, was opened at 2013-02-06 19:14 Message generated for change (Comment added) made by nobody You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3603652...

Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: puredata Group: bugfix Status: Open Resolution: None Priority: 5 Private: No Submitted By: Nobody/Anonymous (nobody) Assigned to: Miller Puckette (millerpuckette) Summary: add bang methods to atan2 and math ops

Initial Comment: Added bang methods to: atan2 cos sin atan tan sqrt log exp abs Added arg to: atan2

----------------------------------------------------------------------

Comment By: Nobody/Anonymous (nobody) Date: 2013-05-21 03:54

Message: Ball Gowns Prom Dresses http://www.dressyus.com/prom-dresses-by-type/ball-gowns-prom-dresses.html <a href="http://www.dressyus.com/prom-dresses-by-type/ball-gowns-prom-dresses.html" title="Ball Gowns Prom Dresses">Ball Gowns Prom Dresses</a>

----------------------------------------------------------------------

Comment By: Hans-Christoph Steiner (eighthave) Date: 2013-02-07 13:10

Message: It makes a lot more sense to add the bang methods where they are missing that remove them. Removing them will break backwards compatibility. Adding them will make things consistent.

----------------------------------------------------------------------

Comment By: IOhannes m zmölnig (zmoelnig) Date: 2013-02-07 12:53

Message: @jonathan: don't remote the bang-methods for now

I'm not in the position to decide what will actually go into Pd, that's miller. it's probably wise to know what he will accept before doing the work (and i doubt that removing the "bang" methods from the unops will ever make it into Pd...just to stay compatible)

----------------------------------------------------------------------

Comment By: Jonathan Wilkes (jancsika1) Date: 2013-02-07 12:42

Message: I'll separate out all the changes to the unary operators into a different patch, then revise the patch here to only add the arg to atan2 and give it a bang method. This will make atan2 consistent with every other binop in arithmetic.c.

I'll also add a patch that removes the bang methods from every single class in x_arithmetic.c.

----------------------------------------------------------------------

Comment By: IOhannes m zmölnig (zmoelnig) Date: 2013-02-07 10:54

Message: true.

but i would go for the "none" route, if somebody asked me :-)

anyhow, i'm not aware of any math "unop" that can be banged (but i haven't dug deep in my memory)

----------------------------------------------------------------------

Comment By: Hans-Christoph Steiner (eighthave) Date: 2013-02-07 07:45

Message: objects with a cold inlet store the last value, by definition, so any object with a cold inlet keeps an internal state. the majority of the math objects with a cold inlet respond to a bang by using that stored value to run the calculation again. Either all of the math objects or none of the math objects should respond to the bang. Its totally arbitrary and inconsistent to have some that respond and some that don't

----------------------------------------------------------------------

Comment By: IOhannes m zmölnig (zmoelnig) Date: 2013-02-07 00:12

Message: i understand the argument to [atan2], but why would you need to [bang( all those messages? i think it is very nice that they don't have an internal state (and actually more objects should not be bangable, just because you can)

----------------------------------------------------------------------

Comment By: Jonathan Wilkes (jancsika1) Date: 2013-02-06 20:53

Message: Well, this patch should cover everything in x_arithmetic.c.

----------------------------------------------------------------------

Comment By: Hans-Christoph Steiner (eighthave) Date: 2013-02-06 20:11

Message: For consistency's sake, all of the math objects should respond to the bang message and accept arguments when they have a second inlet.

----------------------------------------------------------------------

You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3603652...