Opened 12 years ago

Closed 9 years ago

#11426 closed defect (invalid)

point3d needs to accept single vectors, not just lists of them

Reported by: kcrisman Owned by: jason, was
Priority: major Milestone: sage-duplicate/invalid/wontfix
Component: graphics Keywords:
Cc: jason, slabbe Merged in:
Authors: Reviewers: Punarbasu Purkayastha
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

GitHub link to the corresponding issue

Description

sage: point3d([3,2,1])
<works fine>
sage: point3d([vector([3,2,1]),vector([1,2,3])])
<works fine>
sage: point3d(vector(3,2,1))
----------------------------------------------------------
TypeError: object of type 'sage.rings.integer.Integer' has no len()

See also #10948.

Change History (5)

comment:1 Changed 9 years ago by jdemeyer

Milestone: sage-5.11sage-5.12

comment:2 Changed 9 years ago by ppurka

Milestone: sage-5.13sage-duplicate/invalid/wontfix
Status: newneeds_review

Invalid ticket: vector(3,2,1) should be vector([3,2,1])

comment:3 Changed 9 years ago by ppurka

Reviewers: Punarbasu Purkayastha
Status: needs_reviewpositive_review

comment:4 Changed 9 years ago by kcrisman

Huh, you are right! I don't know why I didn't notice that when I first generated this. Such a nice round point in the middle of a Jmol box...

comment:5 Changed 9 years ago by jdemeyer

Resolution: invalid
Status: positive_reviewclosed
Note: See TracTickets for help on using tickets.