Ticket #5450 (closed enhancement: fixed)

Opened 4 years ago

Last modified 4 years ago

[with patch, positive review] plotting a vector as a point plots a sphere, not a point

Reported by: jason Owned by: jason
Priority: major Milestone: sage-3.4.1
Component: graphics Keywords:
Cc: Work issues:
Report Upstream: Reviewers:
Authors: Merged in:
Dependencies: Stopgaps:

Description

Presumably, plotting a point is more efficient. However, 

vector([1,2,3]).plot(plot_type='point')

plots a sphere instead of a point3d (or point2d, if 2-dimensional). This should be changed to plot a point.

Attachments

trac-5450-vector_plot_fast3d.patch Download (2.1 KB) - added by jason 4 years ago.

Change History

comment:1 Changed 4 years ago by jason

  • Owner changed from was to jason
  • Status changed from new to assigned

comment:2 Changed 4 years ago by jason

Another performance enhancement would be using a line3d with arrow_head=True instead of an arrow3d command. I think that would also preserve aspect ratio better anyway.

comment:3 Changed 4 years ago by jason

using the line3d command instead of the arrow3d command I think would be more in line with the result of vector([1,2]).plot().plot3d() (which I believe uses the line3d command).

Changed 4 years ago by jason

comment:4 Changed 4 years ago by jason

  • Summary changed from plotting a vector as a point plots a sphere, not a point to [with patch, needs review] plotting a vector as a point plots a sphere, not a point

This should make plotting lots of 3d vector arrows or points significantly faster.

comment:5 Changed 4 years ago by robertwb

  • Summary changed from [with patch, needs review] plotting a vector as a point plots a sphere, not a point to [with patch, positive review] plotting a vector as a point plots a sphere, not a point

Looks good.

comment:6 Changed 4 years ago by mabshoff

  • Status changed from assigned to closed
  • Resolution set to fixed
  • Milestone changed from sage-3.4.2 to sage-3.4.1

Merged in Sage 3.4.1.rc3.

Cheers,

Michael

Note: See TracTickets for help on using tickets.