Ticket #2645 (closed defect: fixed)

Opened 5 years ago

Last modified 5 years ago

[with patch, with positive review] arrow3d is sometimes too long

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

Description

The patch below cuts off the shaft and part of the head if an arrow3d is too long for the given vector.

Attachments

arrow3d-short.patch Download (1.9 KB) - added by jason 5 years ago.

Change History

Changed 5 years ago by jason

comment:1 Changed 5 years ago by jason

  • Summary changed from arrow3d is sometimes too long to [with patch, needs review] arrow3d is sometimes too long

comment:2 Changed 5 years ago by mabshoff

In  https://groups.google.com/group/sage-devel/browse_thread/thread/d88bc7503638af0c Robert Bradshaw commented: 

The line3d command will produce much faster arrows:

line3d([(0,0,0), (1,2,3)], thickness=2, arrow_head=True)

The ds parameter is supposed to relate the size of the overall scene  
to the number of pixels in the final render. I agree there needs to  
be a better way to set it.

Cheers,

Michael

comment:3 Changed 5 years ago by jason

The comment about line3d does not change the bug and does not change the patch. The comment about line3d was to address a problem with plot_vector_field3d (#2646)

comment:4 Changed 5 years ago by robertwb

  • Summary changed from [with patch, needs review] arrow3d is sometimes too long to [with patch, with positive review] arrow3d is sometimes too long

Works great for me. A good example of this is 

sage: sum([arrow3d((cos(t),sin(t),0),(cos(t),sin(t),t/10)) for t in [0,0.3,..,2*pi]])

comment:5 Changed 5 years ago by mabshoff

  • Status changed from new to closed
  • Resolution set to fixed

Merged in Sage 2.11.alpha2

Note: See TracTickets for help on using tickets.