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

numpy -> vector ignores ring #17423

Closed
fph opened this issue Dec 1, 2014 · 9 comments
Closed

numpy -> vector ignores ring #17423

fph opened this issue Dec 1, 2014 · 9 comments
Labels
c: linear algebra p: minor / 4 t: bug
Milestone
sage-6.5

Comments

@fph
Copy link

fph commented Dec 1, 2014 

sage: import numpy as np
sage: v = np.asarray([0.5,1])
sage: vector(RR,v).parent()
Vector space of dimension 2 over Real Double Field

vv is created with ring RDF rather than the correct one.

Component: linear algebra

Author: Jeroen Demeyer

Branch/Commit: 7a5a35c

Reviewer: Ralf Stephan

Issue created by migration from https://trac.sagemath.org/ticket/17423
@fph fph added this to the sage-6.5 milestone Dec 1, 2014
@jdemeyer
Copy link

jdemeyer commented Dec 2, 2014

comment:1

Workaround:

vv = vector(RealField(rnd='RNDU'),list(v))

@jdemeyer jdemeyer changed the title Wrong ring in vector() numpy -> vector ignores ring Dec 28, 2014
@jdemeyer
Copy link

Author: Jeroen Demeyer

@jdemeyer

This comment has been minimized.

Sign in to view
@jdemeyer
Copy link

Branch: u/jdemeyer/ticket/17423

@jdemeyer
Copy link

Commit: 7a5a35c

@jdemeyer
Copy link

New commits:

7a5a35cDon't ignore ring for numpy -> vector conversion

@rwst
Copy link

rwst commented Jan 2, 2015

comment:7

Looks good, passes make ptestlong.

@rwst
Copy link

rwst commented Jan 2, 2015

Reviewer: Ralf Stephan

@vbraun
Copy link
Member

vbraun commented Jan 2, 2015

Changed branch from u/jdemeyer/ticket/17423 to 7a5a35c

@vbraun vbraun closed this as completed in 610c876 Jan 2, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: linear algebra p: minor / 4 t: bug
Projects
None yet
Milestone
sage-6.5
Development

No branches or pull requests
4 participants
@vbraun @fph @rwst @jdemeyer