Ticket #5635 (closed defect: fixed)

Opened 4 years ago

Last modified 4 years ago

[with patch, positive review] plot method on lattice polytopes gives something ridiculous

Reported by: was Owned by: mhansen
Priority: major Milestone: sage-3.4.1
Component: combinatorics Keywords:
Cc: sage-combinat Work issues:
Report Upstream: Reviewers:
Authors: Merged in:
Dependencies: Stopgaps:

Description

The plot method on an object should return either a 2d plot or raise NotImplementedError? (or not be defined). On LatticePolytope?'s it returns a 3d Tachyon object. 

sage: p = LatticePolytope(random_matrix(ZZ, 3,6, x=7)).plot()
sage: type(p)
<class 'sage.plot.tachyon.Tachyon'>

Attachments

11803.patch Download (3.2 KB) - added by novoselt 4 years ago.

Change History

Changed 4 years ago by novoselt

comment:1 Changed 4 years ago by novoselt

  • Summary changed from plot method on lattice polytopes gives something ridiculous to [with patch, needs review] plot method on lattice polytopes gives something ridiculous

The patch removes plot() method and fixes the documentation. I also changed show() to show3d(), which shows the plot without axes.

comment:2 Changed 4 years ago by was

  • Summary changed from [with patch, needs review] plot method on lattice polytopes gives something ridiculous to [with patch, positive review] plot method on lattice polytopes gives something ridiculous

comment:3 Changed 4 years ago by mabshoff

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

Merged in Sage 3.4.1.rc0.

Cheers,

Michael

comment:4 Changed 4 years ago by nthiery

  • Cc sage-combinat added
Note: See TracTickets for help on using tickets.