bccp / nbodykit

Analysis kit for large-scale structure datasets, the massively parallel way
http://nbodykit.rtfd.io
GNU General Public License v3.0
111 stars 60 forks source link

Odd multipoles -- support for negative mu #608

Closed mehdirezaie closed 4 years ago

mehdirezaie commented 4 years ago

Hi!

I am opening this issue to add support for negative mu to correctly compute odd multipoles in P(k).

Thanks a lot!!

rainwoodman commented 4 years ago

Is this fixed by #610?

mehdirezaie commented 4 years ago

I believe so! @akrolewski is a better person to confirm though since I did not use the code for odd multipoles since the DESI meeting. I remember Alex presented results with DESI mocks showing the code yield similar results to that of other implementations (e.g., Florian Beutler's). So I guess we can close this.

akrolewski commented 4 years ago

Yep, that's right--#610 fixes this and we can close the issue.

rainwoodman commented 4 years ago

Thanks for confirming!