Ticket #6184 (closed defect: fixed)

Opened 10 months ago

Last modified 9 months ago

[with patch, positive review] mesh=True and dots=True don't work for 3D plots

Reported by: wcauchois Owned by: was
Priority: major Milestone:
Component: graphics Keywords:
Cc: Author(s): Bill Cauchois
Report Upstream: Reviewer(s): Marshall Hampton
Merged in: 4.0.1.rc0 Work issues:

Description

Just like the title says; the options noted do not have any effect on 3D plots in Sage 4.0.rc0.

For example, the command

plot3d(lambda x,y: exp(x+y*I).real(), (-2, 2.4), (-3, 3), mesh=True)

Should display a 3D plot with mesh lines drawn in. However, the result does not have mesh lines.

(This bug was discussed at  this forum thread.)

I will attach a patch that fixes the issue.

Attachments

trac6184.patch Download (1.3 KB) - added by wcauchois 10 months ago.
based on sage 4.0.rc0

Change History

Changed 10 months ago by wcauchois

based on sage 4.0.rc0

Changed 10 months ago by wcauchois

  • summary changed from mesh=True and dots=True don't work for 3D plots to [with patch, needs review] mesh=True and dots=True don't work for 3D plots

Does anyone know where this functionality was broken, and what the code was like before?

Changed 10 months ago by mhampton

  • owner changed from tbd to was
  • component changed from algebra to graphics
  • summary changed from [with patch, needs review] mesh=True and dots=True don't work for 3D plots to [with patch, positive review] mesh=True and dots=True don't work for 3D plots

This seems to fix the problem and not cause others in the limited testing I have done. Since it is a surgical-strike type of patch I feel good about giving it a positive review - this is basically two extra lines that correctly pass on an option.

Changed 10 months ago by mhansen

  • status changed from new to closed
  • resolution set to fixed

Merged in 4.0.1.rc0.

Changed 9 months ago by mvngu

  • reviewer set to Marshall Hampton
  • merged set to 4.0.1.rc0
  • author set to Bill Cauchois
Note: See TracTickets for help on using tickets.