Opened 5 years ago

Last modified 5 years ago

#22631 needs_work enhancement

Allow plotting annulus/ring

Reported by: kcrisman Owned by: asutosh7hota
Priority: major Milestone: sage-8.0
Component: graphics Keywords:
Cc: Merged in:
Authors: Reviewers:
Report Upstream: N/A Work issues:
Branch: u/asutosh7hota/allow_plotting_annulus_ring (Commits, GitHub, GitLab) Commit: 7e233a26f05695306a5bd66c0d78eee21c76450d
Dependencies: Stopgaps:

Status badges


It turns out that the matplotlib patch we use for Disk supports plotting an annulus/ring as well. See this example page and the render on subplot method for Disk where this could easily be instituted. We'd need the documentation to make it very clear one can get a (partial) ring/annulus this way.

Change History (5)

comment:1 Changed 5 years ago by kcrisman

(See also #22615.)

comment:2 Changed 5 years ago by asutosh7hota

  • Owner changed from (none) to asutosh7hota

I am going throug the docs that you mentioned. I hope I would be able to add a couple of examples implementing a partial ring.

Last edited 5 years ago by asutosh7hota (previous) (diff)

comment:3 Changed 5 years ago by asutosh7hota

  • Branch set to u/asutosh7hota/allow_plotting_annulus_ring

comment:4 Changed 5 years ago by asutosh7hota

  • Commit set to 7e233a26f05695306a5bd66c0d78eee21c76450d
  • Status changed from new to needs_review

I have added an example as mentioned in the ticket description. Kindly review and suggest changes

New commits:

7e233a2Examples added for #22631

comment:5 Changed 5 years ago by kcrisman

  • Status changed from needs_review to needs_work

What I meant was to modify the Disk function to allow for using this functionality already built in the matplotlib patch. This example would make more sense on #22615.

Note: See TracTickets for help on using tickets.