[ESSREDUCE] Move esspolarization to essreduce

[jokasimr]

As suggested in #276

[nvaytet]

Suggested change

# ruff: noqa: E402, F401, I

Not sure this is needed?

[nvaytet]

I guess this should go away? It should just use the version of essreduce.

[nvaytet]

Maybe we can add a docstring here for the polarization submodule that describes a little the contents?

[nvaytet]

I think you are missing the docs contents of esspolarization? e.g. https://scipp.github.io/esspolarization/user-guide/sans-polarization-analysis-methodology.html

That said, I don't know how much of it we want to keep, but I would say at least to start we would want to keep all of it?

[jokasimr]

Yes that's true, I forgot to move that 👍

[nvaytet]

I think you should also add polarization in the top level init ?

[jokasimr]

Although... Doing it this way we loose the history of the essreduceesspolarization package. That's not good. Maybe it's better to merge #276 instead and then do the move after?

[YooSunYoung]

Although... Doing it this way we loose the history of the essreduce package. That's not good. Maybe it's better to merge #276 instead and then do the move after?

You mean the history of the esspolarization?
I vote for merging it first and then move...
then it's more clear who contributed to the code and that we moved the package into the essreduce.

In that case, in #276, we don't need to update pixi.toml or readme since we know that it'll be merged into essreduce.

[nvaytet]

Maybe it's better to merge #276 instead and then do the move after?

Yes! do that 🙂