Opened 10 years ago
Closed 10 years ago
#13009 closed enhancement (fixed)
Add blank lines for a couple plots in reference manual
Reported by: | kcrisman | Owned by: | jason, was |
---|---|---|---|
Priority: | trivial | Milestone: | sage-5.1 |
Component: | graphics | Keywords: | beginner, sd40.5 |
Cc: | Merged in: | sage-5.1.beta4 | |
Authors: | Dan Drake | Reviewers: | Karl-Dieter Crisman |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description
In graphics.py, we have
plot(sin(pi*x), (x, -8, 8)) # Labels not so helpful plot(sin(pi*x), (x, -8, 8), ticks=2) # Multiples of 2 plot(sin(pi*x), (x, -8, 8), ticks=[[-7,-3,0,3,7],[-1/2,0,1/2]]) # Your choices plot(sin(pi*x), (x, -8, 8), ticks=[[],[]]) # No ticks at all!
but then in the live documentation only the last one shows up, so we should separate them with
::
things.
Attachments (1)
Change History (6)
comment:1 Changed 10 years ago by
comment:2 Changed 10 years ago by
Well, whatever. We've basically been doing it this way for a while, and since plot(foo)
does indeed implicitly call show, it's a little easier. There are some doctests out there that use the show()
syntax. But I do prefer the ::
business.
Changed 10 years ago by
comment:3 Changed 10 years ago by
- Status changed from new to needs_review
comment:4 Changed 10 years ago by
- Reviewers set to Karl-Dieter Crisman
- Status changed from needs_review to positive_review
I don't see a problem with this. Positive review, thanks.
comment:5 Changed 10 years ago by
- Merged in set to sage-5.1.beta4
- Resolution set to fixed
- Status changed from positive_review to closed
Note: See
TracTickets for help on using
tickets.
Oh, is that why there are so many extra
::
in Sage docstrings? I wondered about that.Why not just wrap them with show() or put them in a single "line" by wrapping them with parentheses and semicolons?
or