Re: Interesting failure of Collect

*To*: mathgroup at smc.vnet.net*Subject*: [mg61344] Re: [mg61307] Interesting failure of Collect*From*: Andrzej Kozlowski <akoz at mimuw.edu.pl>*Date*: Sun, 16 Oct 2005 00:17:59 -0400 (EDT)*References*: <200510150222.WAA17294@smc.vnet.net> <763FAE8A-8F0D-4BB1-A438-DD558C416369@mimuw.edu.pl>*Sender*: owner-wri-mathgroup at wolfram.com

On 15 Oct 2005, at 15:34, Andrzej Kozlowski wrote: > > On 15 Oct 2005, at 11:22, Blimbaum, Jerry CIV NSWC PC wrote: > > >> >> Given the expr = Sin[p]/(4 Pi a) + Sin[p]/(4 Pi b) >> >> and then applying the command >> >> >> Collect[expr, Sin[p]/(4 Pi)] works as it should, however, >> >> expr2 = Sin[p]/(4 Pi a) - Sin[p]/(4 Pi b) >> >> leaves the expression untouched. (I realize I could use Simplify >> but >> this problem occurred on a much longer expression and this is >> intended >> just to convey the basic idea)....for longer expressions the Collect >> process works for all quantities with a + sign but the one with a >> minus >> sign will not be collected.....strikes me as a "bug"..... >> >> >> thanks....jerry blimbaum >> >> >> >> > > This looks like the standard problem with Mathematica FullForm of > expressions, or more precisely with the way rational numbers are > represented. To see what I mean just compare: > > > FullForm[a-3 b/4] > > > Plus[a,Times[Rational[-3,4],b]] > > > FullForm[a+3 b/4] > > > Plus[a,Times[Rational[3,4],b]] > > > You see that one involves Rational[3,4] and the other Rational > [-3,4] so a pattern involving 3/4 (Rational[3,4]) will only match > the first. So in your case you could do something like this: > > > Collect[expr /. Rational[p_,x_] :> f[p]*Rational[1,x], Sin[p]/ > (4*Pi)] /. f -> Identity > > > ((1/b + 1/a)*Sin[p])/(4*Pi) > > > Collect[expr2 /.Rational[p_, x_] :>f[p]*Rational[1, x],Sin[p]/ > (4*Pi)] /. f -> Identity > > > ((1/a - 1/b)*Sin[p])/(4*Pi) > > > I should add that it is not entirely clear if Collect simply relies > on pattern matching but it seems to suffer from the same problems > as pattern matching. One imagines that by using something like the > above workaround it should be possible to make Collect more robust, > so I tend to agree with you that this may be called a "bug" - in > quotation marks of course ;-) . > > Andrzej Kozlowski > Tokyo, Japan > On further reflection I wish to withdraw my last remark. I do not think this is a "bug", even if the quotation marks are meant to suggest poor design or judgement rather than a programming error. The point is that, as the documentation tells you: Collect[expr, x] collects together terms involving the same powers of object matching x. Obviously the key word here is "Powers of x". This means that Collect is only going to work in general for certain types of x. To illustrate what I mean consider the expression: p = 1 + y*a + y*b + y^2*c + y^2*d; we can collect with respect to y: Collect[p, y] (c + d)*y^2 + (a + b)*y + 1 O.K., now let's substitute 2 x for y: q = p /. y -> 2*x 4*c*x^2 + 4*d*x^2 + 2*a*x + 2*b*x + 1 Now, even though this is essentially the same polynomial Collecting with respect to 2x will not work as before: Collect[q, 2*x] 4*c*x^2 + 4*d*x^2 + 2*(a + b)*x + 1 Collect[q, x] (4*c + 4*d)*x^2 + (2*a + 2*b)*x + 1 The point I am making is that Collect[expr, Sin[p]/(4 Pi)] is only going to work in the relatively unimportant linear expressions in Sin[p]/(4 Pi), while Collect is actually intended for situations involving higher powers. And of course if you collect only with respect to Sin[p] rather than Sin[p]/(4*Pi) then your problem does not arise: Collect[Sin[p]/(4*Pi*a) - Sin[p]/(4*Pi*b), Sin[p]] (1/(4*a*Pi) - 1/(4*b*Pi))*Sin[p] Collect[Sin[p]/(4*Pi*a) + Sin[p]/(4*Pi*b), Sin[p]] (1/(4*b*Pi) + 1/(4*Pi*a))*Sin[p] Another illustration of this point: note that an extension of your example no longer works properly in this non-linear case: Collect[Sin[p]/(4*Pi*a) + (Sin[p]/(4*Pi*a))^2 + Sin[p]/(4*Pi*b) + (Sin[p]/(4*Pi*b))^2, Sin[p]/(4*Pi)] Sin[p]^2/(16*a^2*Pi^2) + Sin[p]^2/(16*b^2*Pi^2) + ((1/b + 1/a)*Sin[p])/(4*Pi) Although of course this works: Collect[Sin[p]/(4*Pi*a) + (Sin[p]/(4*Pi*a))^2 + Sin[p]/(4*Pi*b) + (Sin[p]/(4*Pi*b))^2, Sin[p]] (1/(16*b^2*Pi^2) + 1/(16*Pi^2*a^2))*Sin[p]^2 + (1/(4*b*Pi) + 1/(4*Pi*a))*Sin[p] So no, I do not agree with you. Collect[..., Sin[p]/(4 Pi)] is not the intended usage of Collect, it works in a special case by accident rather than by design. Collecting linear expressions is not important enough to warrant any changes in Collect to make it work as you think it should. Making it work with non linear expressions would require a completely different approach to pattern matching. So I conclude that neither a bug nor even a "bug" is involved here. Andrzej Kozlowski Tokyo, Japan

**References**:**Interesting failure of Collect***From:*"Blimbaum, Jerry CIV NSWC PC" <jerry.blimbaum@navy.mil>

**Re: Re: How to reverse sign on the y's in a list of (x, y)'s?**

**Getting a pure text widget?**

**Re: Interesting failure of Collect**

**Re: Interesting failure of Collect**