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

more types for sage_input: vectors, matrices, etc. #5101

Closed
sagetrac-cwitty mannequin opened this issue Jan 25, 2009 · 3 comments
Closed

more types for sage_input: vectors, matrices, etc. #5101

sagetrac-cwitty mannequin opened this issue Jan 25, 2009 · 3 comments

Comments

@sagetrac-cwitty
Copy link
Mannequin

sagetrac-cwitty mannequin commented Jan 25, 2009

I'm attaching a patch to increase sage_input support. Newly supported are floats, elements of RDF and CC, vectors, and matrices (sparse and dense). I also added some new features to sage_input for handling these new types, and fixed a bug in matrix_modn_sparse that was exposed in the process (sorry, the bug fix really should have been a separate patch).

This patch depends on #2898 (and hence indirectly on #3938).

CC: @jasongrout

Component: user interface

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

@sagetrac-cwitty sagetrac-cwitty mannequin added this to the sage-3.4.1 milestone Jan 25, 2009
@sagetrac-cwitty sagetrac-cwitty mannequin self-assigned this Jan 25, 2009
@jasongrout
Copy link
Member

comment:1

Attachment: sage-input-sequel.patch.gz

I've looked over the patch and it looks reasonable. Doctests pass on 3.3alpha3 for all (nontrivially) touched files. Positive review.

This does depend on someone reviewing 2898, though.

@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Feb 8, 2009

comment:2

This fix depends on #2898 getting a positive review and getting merged.

Cheers,

Michael

@sagetrac-mabshoff sagetrac-mabshoff mannequin modified the milestones: sage-3.4.1, sage-3.3 Feb 8, 2009
@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Feb 9, 2009

comment:3

Merged in Sage 3.3.rc0.

Cheers,

Michael

@sagetrac-mabshoff sagetrac-mabshoff mannequin closed this as completed Feb 9, 2009
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

1 participant