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.

Sign up for GitHub

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

Jump to bottom

Manage subdivisions when augmenting matrices #10425

Closed
rbeezer mannequin opened this issue Dec 3, 2010 · 6 comments
Closed

Manage subdivisions when augmenting matrices #10425

rbeezer mannequin opened this issue Dec 3, 2010 · 6 comments
Assignees
@jasongrout @williamstein
Labels
c: linear algebra r: duplicate t: enhancement

Comments

@rbeezer
Copy link
Mannequin

rbeezer mannequin commented Dec 3, 2010

Patch will allow optional recording of a new matrix subdivision when augmenting two matrices.

What should happen with row subdivisions? The solution here is to preserve them (if a subdivided matrix as a result is requested) if and only if the two matrices have equal row subdivisions.

After this is reviewed, I'll implement similar behavior for the stack() command.

Patch moved to #10424.

CC: @jdemeyer

Component: linear algebra

Reviewer: Rob Beezer

Issue created by migration from https://trac.sagemath.org/ticket/10425
@rbeezer rbeezer mannequin added this to the sage-4.6.1 milestone Dec 3, 2010
@rbeezer rbeezer mannequin added c: linear algebra labels Dec 3, 2010
@rbeezer rbeezer mannequin assigned jasongrout and williamstein Dec 3, 2010
@rbeezer
Copy link
Mannequin Author

rbeezer mannequin commented Dec 3, 2010

Attachment: trac_10425-matrix-augment-subdivide.patch.gz

@rbeezer rbeezer mannequin added the s: needs review label Dec 3, 2010
@rbeezer
Copy link
Mannequin Author

rbeezer mannequin commented Dec 7, 2010

comment:2

#10424 needed reworking and I've moved this code there. Once that ticket is closed, I'll make sure this one gets banished.

@rbeezer rbeezer mannequin added s: needs info and removed s: needs review labels Dec 7, 2010
@rbeezer
Copy link
Mannequin Author

rbeezer mannequin commented Dec 14, 2010

comment:3

Release Manager

#10424 now has a positive review, so this can be marked obsolete whenever it is appropriate.

@jdemeyer
Copy link

Reviewer: Rob Beezer

@jdemeyer
Copy link

Changed author from Rob Beezer to none

@jdemeyer

This comment has been minimized.

Sign in to view
@jdemeyer jdemeyer removed this from the sage-4.6.1 milestone Dec 14, 2010
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jasongrout jasongrout

@williamstein williamstein

Labels
c: linear algebra r: duplicate t: enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@jasongrout @williamstein @jdemeyer