Currently, the public API for SymmetricTensor only includes symmetric_axes, which is confusing because we also have support for arbitrary permutation groups.
I think we should:
- Deprecate symmetric_axes
- Unify the nomenclature around
symmetries/groups/symmetry_groups, which I think are all the same thing
- Allow passing partitions of indices to
symmetries, which are automatically cast to the corresponding Young group. (I.e., this should match the old behavior of symmetric_axes)
- Make sure
symmetries and not symmetric_axes is used for all FLOP counting
Currently, the public API for
SymmetricTensoronly includessymmetric_axes, which is confusing because we also have support for arbitrary permutation groups.I think we should:
symmetries/groups/symmetry_groups, which I think are all the same thingsymmetries, which are automatically cast to the corresponding Young group. (I.e., this should match the old behavior ofsymmetric_axes)symmetriesand notsymmetric_axesis used for all FLOP counting