Impeller matrices use column-major storage and the constructor expect arguments in the same order. This is confusing to API users who may be used to arguments to be in row-major storage order. Just have named constexpr factories so this is easier to reason about. Perhaps even remove the current constructors (likely to be a larger change now though).