Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#6184 closed defect (fixed)

[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: Merged in: 4.0.1.rc0
Authors: Bill Cauchois Reviewers: Marshall Hampton
Report Upstream: Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

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 (1)

trac6184.patch (1.3 KB) - added by wcauchois 12 years ago.
based on sage 4.0.rc0

Download all attachments as: .zip

Change History (5)

Changed 12 years ago by wcauchois

based on sage 4.0.rc0

comment:1 Changed 12 years 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?

comment:2 Changed 12 years ago by mhampton

  • Component changed from algebra to graphics
  • Owner changed from tbd to was
  • 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.

comment:3 Changed 12 years ago by mhansen

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

Merged in 4.0.1.rc0.

comment:4 Changed 12 years ago by mvngu

  • Authors set to Bill Cauchois
  • Merged in set to 4.0.1.rc0
  • Reviewers set to Marshall Hampton
Note: See TracTickets for help on using tickets.