`MatrixMorphism.matrix`: Change meaning/default of `side`

sagemath / sage

Main repository of SageMath. Now open for Issues and Pull Requests.

https://www.sagemath.org

Other

1.08k stars 394 forks source link

`MatrixMorphism.matrix`: Change meaning/default of `side` #37876

Open mkoeppe opened 2 weeks ago

mkoeppe commented 2 weeks ago

Partially addresses #37877 -- by making a breaking change to what side means in MatrixMorphism.matrix to matchFiniteDimensionalModulesWithBasis.MorphismMethods.matrix.

:memo: Checklist

[x] The title is concise and informative.
[x] The description explains in detail what this PR is about.
[ ] I have linked a relevant issue or discussion.
[ ] I have created tests covering the changes.
[ ] I have updated the documentation and checked the documentation preview.

:hourglass: Dependencies

github-actions[bot] commented 2 weeks ago

Documentation preview for this PR (built with commit e48ce451058550392c7932d22d2ef253256b4a6e; changes) is ready! :tada: This preview will update shortly after each push to this PR.