-
Notifications
You must be signed in to change notification settings - Fork 58
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
similar/zero can still generate matrices of the wrong type #1045
Comments
Or better yet it should ignore the ring of the first argument and look at the type of the Ring argument instead. |
You guys don't define custom polynomial types that don't use generic polynomials in Hecke do you? |
No. |
(And I agree with your conclusion.) |
Apparently we decided in #651 that this was desired behaviour, so as to not break deepcopy and related things. In fact, the regression test breaks if I make the change that I think makes sense. So I'm going to leave this one alone. |
I think the Nemo version of similar has to do the coercion into the new ring itself, rather than rely on AbstractAlgebra to do it, as the latter will always generate a generic matrix.
The text was updated successfully, but these errors were encountered: