Allow is_transpose_needed to work on std::size_t based array

[yasahi-hpc]

This PR aims at generalizing is_transpose_needed helper.
It may be that the input type is std::array<std::size_t, DIM> for distributed case

[science-enthusiast]

Looks fine to me. Just one observation. There is no test for static_assert(std::is_integral_v<IntType>, "is_transpose_needed: IntType must be an integral type.");. Maybe it is too much work to add an invalid value_type to the list of possible value_types.

[yasahi-hpc]

@science-enthusiast
Thanks for the review

Looks fine to me. Just one observation. There is no test for static_assert(std::is_integral_v, "is_transpose_needed: IntType must be an integral type.");. Maybe it is too much work to add an invalid value_type to the list of possible values.

Strictly speaking, we can parameterize over more types: int8_t and so on.
In practice, we only use either int or std::size_t, so current tests should be fine

[PaulGannay]

Seems fine otherwise

[PaulGannay]

IntType should be renamed to IndexType.

[yasahi-hpc]

Thanks
I have fixed accordingly

[yasahi-hpc]

Thanks @science-enthusiast @PaulGannay

I will merge this