Skip to content
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

Remove deprecated module sage.matrix.matrix and method Matrix_integer_dense.__invert_unit #32417

Closed
mkoeppe opened this issue Aug 24, 2021 · 16 comments

Comments

@mkoeppe
Copy link
Member

mkoeppe commented Aug 24, 2021

sage.matrix.matrix was deprecated 4 years ago in #24096.

Matrix_integer_dense.__invert_unit was deprecated 3 years ago in #25084.

We remove them.

CC: @videlec @slel @tscrim

Component: refactoring

Author: Matthias Koeppe

Branch/Commit: cebc952

Reviewer: Travis Scrimshaw

Issue created by migration from https://trac.sagemath.org/ticket/32417

@mkoeppe mkoeppe added this to the sage-9.5 milestone Aug 24, 2021
@mkoeppe
Copy link
Member Author

mkoeppe commented Aug 24, 2021

@mkoeppe
Copy link
Member Author

mkoeppe commented Aug 24, 2021

Author: Matthias Koeppe

@mkoeppe

This comment has been minimized.

@mkoeppe
Copy link
Member Author

mkoeppe commented Aug 24, 2021

New commits:

f902733sage.matrix.matrix: Remove deprecated module
9f8e5e3Matrix_integer_dense.__invert_unit: Remove deprecated method

@mkoeppe
Copy link
Member Author

mkoeppe commented Aug 24, 2021

Commit: 9f8e5e3

@mkoeppe mkoeppe changed the title Remove deprecated module sage.matrix.matrix Remove deprecated module sage.matrix.matrix and method Matrix_integer_dense.__invert_unit Aug 24, 2021
@tscrim
Copy link
Collaborator

tscrim commented Aug 25, 2021

Reviewer: Travis Scrimshaw

@tscrim
Copy link
Collaborator

tscrim commented Aug 25, 2021

comment:3

LGTM.

@mkoeppe
Copy link
Member Author

mkoeppe commented Aug 25, 2021

comment:4

Thanks!

@vbraun
Copy link
Member

vbraun commented Sep 14, 2021

comment:5

Merge conflict

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Sep 14, 2021

Changed commit from 9f8e5e3 to cebc952

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Sep 14, 2021

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

bcc2397sage.matrix.matrix: Remove deprecated module
cebc952Matrix_integer_dense.__invert_unit: Remove deprecated method

@mkoeppe
Copy link
Member Author

mkoeppe commented Sep 14, 2021

comment:7

trivial rebase

@kiwifb
Copy link
Member

kiwifb commented Sep 17, 2021

comment:8

I get quite a lot of failure building sage since Volker has put it in its develop branch to test merging this morning

Error compiling Cython file:
------------------------------------------------------------
...

from .encoder import Encoder
from .decoder import Decoder, DecodingError

from sage.modules.free_module_element cimport FreeModuleElement
from sage.matrix.matrix cimport Matrix
^
------------------------------------------------------------

sage/coding/ag_code_decoders.pyx:73:0: 'sage/matrix/matrix.pxd' not found

Error compiling Cython file:
------------------------------------------------------------
...

from .encoder import Encoder
from .decoder import Decoder, DecodingError

from sage.modules.free_module_element cimport FreeModuleElement
from sage.matrix.matrix cimport Matrix
^
------------------------------------------------------------

sage/coding/ag_code_decoders.pyx:73:0: 'sage/matrix/matrix/Matrix.pxd' not found

and associated errors that Matrix is not defined. The above error is found across numerous files.

@mkoeppe
Copy link
Member Author

mkoeppe commented Sep 17, 2021

comment:9

Yes, looks like something went wrong here. Incremental builds didn't catch this.

@mkoeppe
Copy link
Member Author

mkoeppe commented Sep 17, 2021

comment:10

Fixed in #32530.

@vbraun
Copy link
Member

vbraun commented Sep 19, 2021

Changed branch from u/mkoeppe/remove_deprecated_module_sage_matrix_matrix to cebc952

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants