#17837 closed enhancement (fixed)
Remove matrix_modn_dense.pyx
Reported by:  Jeroen Demeyer  Owned by:  

Priority:  major  Milestone:  sage6.6 
Component:  linear algebra  Keywords:  
Cc:  Merged in:  
Authors:  Jeroen Demeyer  Reviewers:  Vincent Delecroix 
Report Upstream:  N/A  Work issues:  
Branch:  766f1a5 (Commits, GitHub, GitLab)  Commit:  
Dependencies:  Stopgaps: 
Description (last modified by )
This has been unused since #4260, remove it.
Change History (18)
comment:1 Changed 8 years ago by
Authors:  → André Apitzsch 

comment:2 Changed 8 years ago by
Branch:  → u/jdemeyer/ticket/17837 

Created:  Feb 23, 2015, 11:23:05 AM → Feb 23, 2015, 11:23:05 AM 
Modified:  Feb 23, 2015, 11:23:43 AM → Feb 23, 2015, 11:23:43 AM 
comment:3 Changed 8 years ago by
Commit:  → e7681685dc628352e7f0b8d63bffd8fb059897aa 

Status:  new → needs_review 
comment:4 Changed 8 years ago by
Status:  needs_review → needs_work 

comment:5 Changed 8 years ago by
Description:  modified (diff) 

comment:6 Changed 8 years ago by
Authors:  André Apitzsch → Jeroen Demeyer 

Description:  modified (diff) 
Summary:  Remove redundancies from matrix_modn_dense.pyx → Remove matrix_modn_dense.pyx 
comment:7 Changed 8 years ago by
Commit:  e7681685dc628352e7f0b8d63bffd8fb059897aa → 74d2c66c1955fc7b73e24e0203686e95398ee501 

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

comment:8 Changed 8 years ago by
Commit:  74d2c66c1955fc7b73e24e0203686e95398ee501 → a6112cec279b21c9ae6db3d4a912986693d699de 

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

comment:9 Changed 8 years ago by
Commit:  a6112cec279b21c9ae6db3d4a912986693d699de → 766f1a5a00018c9317cead9dd588d91f17676c68 

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

comment:10 Changed 8 years ago by
Status:  needs_work → needs_review 

comment:11 followup: 16 Changed 8 years ago by
Reviewers:  → Vincent Delecroix 

Status:  needs_review → positive_review 
Instead of
if not (isinstance(b, Matrix_modn_dense_float) or isinstance(b, Matrix_modn_dense_double)):
you might have used
if not isinstance(b, (Matrix_modn_dense_float, Matrix_modn_dense_double)):
which is handled in the exact same way by cython.
I will try to use the nice unpickle_override for #17824.
comment:13 Changed 8 years ago by
Branch:  u/jdemeyer/ticket/17837 → 766f1a5a00018c9317cead9dd588d91f17676c68 

Resolution:  → fixed 
Status:  needs_work → closed 
comment:14 Changed 8 years ago by
Commit:  766f1a5a00018c9317cead9dd588d91f17676c68 

What is this transition "needs work > closed"!?
comment:15 followup: 17 Changed 8 years ago by
On the other hand, I don't understand why the mere existence of ticket #10734 would be a reason for needs_work.
comment:16 Changed 8 years ago by
Replying to vdelecroix:
Instead of
if not (isinstance(b, Matrix_modn_dense_float) or isinstance(b, Matrix_modn_dense_double)):you might have used
if not isinstance(b, (Matrix_modn_dense_float, Matrix_modn_dense_double)):which is handled in the exact same way by cython.
Obviously, I just didn't think of that. It doesn't really matter anyway...
comment:17 followup: 18 Changed 8 years ago by
Replying to jdemeyer:
On the other hand, I don't understand why the mere existence of ticket #10734 would be a reason for needs_work.
Because there is a large amount of work on ticket #10734 that uses matrix_modn_dense
. Before throwing away this work, it would have been nice to wait for the current status of this work. I am pretty sure that everything can be achieved with the LinBox classes anyway.
comment:18 Changed 8 years ago by
Replying to vdelecroix:
Because there is a large amount of work on ticket #10734 that uses
matrix_modn_dense
.
But with or without #17837, matrix_modn_dense
is deprecated and no longer used. So if #10734 really uses Matrix_modn_dense
, it should be changed anyway.
Do we even need the
determinant
method in the first place? Why not rely on inheritance?New commits:
fix some cython warnings