Conversation
| (* f_2^bcad f^ace = - N/2 f^bde *) | ||
| colorSimplifyGeneric[rest_. sunf2[b_, a_SUNIndex, c_SUNIndex, d_] sunf[r1___,a_SUNIndex,r2___,c_SUNIndex,r3___]] := | ||
| (* f_2^bcad f^ace = N/2 f^bde *) | ||
| colorSimplifyGeneric[rest_. sunf2[b_, c_SUNIndex, a_SUNIndex, d_] sunf[r1___,a_SUNIndex,r2___,c_SUNIndex,r3___]] := |
There was a problem hiding this comment.
This is the actual bug fix. Looks like a copy paste error from the case above. All other changes are just to sync the comment with the actual rule
There was a problem hiding this comment.
Looks good. Can you please also add a couple of unit tests to
https://github.com/FeynCalc/feyncalc/blob/master/Tests/SUN/SUNSimplify.test
that cover this case?
There was a problem hiding this comment.
Sure, I added some more unit tests. They should now cover every case at least once.
|
Thanks! It's merged now, even though GH doesn't recognize it. I guess it's because I changed the last commit message to have a more meaningful error description. In any case, thanks for taking time to fix something yourself and I hope that this won't be your last PR to FeynCalc :) |
2 participants
This should fix a wrong replacement rule in colorSimplifyGeneric and resolve the bug described in #368