Possible mistake in documentation regarding storage scheme of SO(3) samples

[PhilippMisofCH]

The documentation of the sampling.so3_samples.f_shape function claims that the storage scheme is $\beta, \alpha, \gamma$

s2fft/s2fft/sampling/so3_samples.py

Lines 16 to 17 in 7cf80bb

	Importantly, the convention adopted for storage of f is :math:`[\beta, \alpha,
	\gamma]`, for Euler angles :math:`(\alpha, \beta, \gamma)` following the

but it returns (gamma, beta, alpha) as far as I see,

s2fft/s2fft/sampling/so3_samples.py

Line 43 in 7cf80bb

return _ngamma(N), _nbeta(L, sampling), _nalpha(L, sampling)

which also seems to be consistent with the rest of the library. Is this a mistake in the documentation or did I misunderstand something?

Thanks in advance for the clarification!

[matt-graham]

Hi @PhilippMisofCH, thanks for flagging this, agreed this looks like a mistake in the documentation (or if not something that needs clarifying!).

@CosmoMatt it looks like the ordering in the returned shape was permuted in 0339538 as part of #134 but the docstring wasn't updated then. Assuming the correct fix is therefore to update the docstring to be consistent the ordering now asssumed in the code?

[jasonmcewen]

Ok, so the documentation indeed just needs updating.

This is a very simply revision if anyone wants to pick it up.