Opened 21 months ago

Closed 21 months ago

#30413 closed defect (fixed)

plot() fails for vertical or horizontal lines.

Reported by: klee Owned by:
Priority: minor Milestone: sage-9.2
Component: graphics Keywords:
Cc: Merged in:
Authors: Kwankyu Lee Reviewers: Marc Mezzarobba
Report Upstream: N/A Work issues:
Branch: d83100d (Commits, GitHub, GitLab) Commit: d83100da9c764dd08a4124e40c10c1687825e4b1
Dependencies: Stopgaps:

Status badges

Description

This works 

sage: R.<x,y> = PolynomialRing(QQ,2)                                                                                                                                 
sage: I = R.ideal([y - x])                                                                                                                                           
sage: I.plot()

but this fails 

sage: R.<x,y> = PolynomialRing(QQ,2)
sage: I = R.ideal([x - 1])
sage: I.plot()

Change History (8)

comment:1 Changed 21 months ago by klee

  • Branch set to u/klee/30413
  • Summary changed from The plot method `I.plot()` fails if `I` defines a vertical or horizontal line. to The plot method I.plot() fails if I defines a vertical or horizontal line.

comment:2 Changed 21 months ago by klee

  • Summary changed from The plot method I.plot() fails if I defines a vertical or horizontal line. to plot() fails for vertical or horizontal lines.

comment:3 Changed 21 months ago by git

  • Commit set to f7475047a69981fbb615beeee9bc393fac2b2020

Branch pushed to git repo; I updated commit sha1. New commits:

​f747504Fix plot() to work for vertical lines

comment:4 Changed 21 months ago by git

  • Commit changed from f7475047a69981fbb615beeee9bc393fac2b2020 to d83100da9c764dd08a4124e40c10c1687825e4b1

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

​d83100dFix plot() to work for vertical lines

comment:5 Changed 21 months ago by klee

  • Status changed from new to needs_review

comment:6 Changed 21 months ago by mmezzarobba

  • Authors set to Kwankyu Lee
  • Reviewers set to Marc Mezzarobba
  • Status changed from needs_review to positive_review

comment:7 Changed 21 months ago by klee

Great. Thank you!

comment:8 Changed 21 months ago by vbraun

  • Branch changed from u/klee/30413 to d83100da9c764dd08a4124e40c10c1687825e4b1
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.