Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

let's release this thing ! #12835

Merged
merged 104 commits into from
Sep 4, 2015
Merged

let's release this thing ! #12835

merged 104 commits into from
Sep 4, 2015

Conversation

carnaval
Copy link
Contributor

everyone cheer, Jake is near the finish line

  • Rendering of Mmap.*
  • Sphinx search result/permalink of Base.process_messages
  • Sphinx search result for macros (e.g. @assert @nloops)
  • Stdlib doc for the doc system?
  • @repl documentation for macrocalls

@tkelman
Copy link
Contributor

tkelman commented Aug 28, 2015

This looks okay on formatting fixes but there are still lingering doc content problems (#12775 (comment)) not addressed yet.

@ViralBShah
Copy link
Member

Can I merge #12766 before this one, or should we wait?

@StefanKarpinski
Copy link
Member

#12766 is a bug fix and can happen any time.

@carnaval
Copy link
Contributor Author

So, when Tony said "lingering docs problem", I'm not sure we realized how bad it was.
This PR now addresses most of what we could find that is critical for having a decent manual. Many of the docstrings are still rst so somewhat poor REPL experience.

In particular :

  • fix various missing docs
  • associate function docstring to tuple types instead of methods so that you can document many methods at once with a wider tuple (the REPL help matching function is still a bit rough and could use some love)
  • separate docstrings in helpdb so that each map to a single rst stdlib entry (a lot of signatures are not the best, could use people going through to look)

A lot of this was done by hand. I think this should make us question our release policy. We basically got nothing out of this doc transition and had to rush it.

I want my last 12 hours back. And I'm pretty sure Julia Computing owes us a bottle.
Never again...

@jakebolewski
Copy link
Member

Drinking bourbon for 12 hours straight is also not healthy.

sort!(fd.order, lt = (a,b) -> ccall(:jl_type_morespecific, Int32, (Any,Any), a, b) > 0)
if haskey(ENV, "GO_AWAY") && haskey(fd.meta, sig) # temporary
error("Adding doc to $f($sig) forbidden GO AWAY")
end
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It does look like you were having fun.

.. code-block:: julia

whos([Module,] [pattern::Regex])
whos([io,] [Module,] [pattern::Regex])
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should probably be an additional signature instead of a code block.

@jakebolewski
Copy link
Member

Documentation is currently broken for the following methods:

(Please add to this list as you discover them)

  • ldltfact
  • Year
  • Month
  • Week
  • Day
  • Hour
  • Minute
  • Second
  • Millisecond
  • @__FILE__
  • scale
  • scale!
  • whos
  • connect (Also duplicated in both Tasks and I/O)
  • unsafe_copy!
  • @sprintf
  • sizeof (Appears in both Essentials and Strings and both are broken)
  • functionloc
  • ones
  • sumabs
  • sumabs!
  • symdiff!
  • middle
  • quantile
  • collect
  • maximum
  • minimum
  • schurfact
  • svd
  • linreg
  • PipeBuffer
  • readdlm (Not sure this one is really broken but it looks like pages of repeated doc for each individual methods - yes it is broken, there's some mismatched double backticks that confused things)
  • sort
  • round
  • svdvals

The following methods are missing synonyms/signatures (Basically @tkelman's comment in 0636e16)

  • Too many to list here, should be all fixed.

The following methods are currently missing documentation:

  • @__LINE__

Other issues (not regressions):

  • ordschur/ordschur! (The doc looks fine but the order is a little wierd ordschur ordschur! ordschur ordschur! instead of ordschur ordschur ordschur! ordschur!) (There's also some other functions in Linear Algebra that has the same issue)

@yuyichao
Copy link
Contributor

yuyichao commented Sep 1, 2015

There's some missing code block quoting in Base.Cartesian Fixed.

@yuyichao
Copy link
Contributor

yuyichao commented Sep 1, 2015

There's a mysterious - _man-packages: in the FAQ above Packages and Modules Fixed.

@yuyichao
Copy link
Contributor

yuyichao commented Sep 1, 2015

The first "results in" for async io should probably have a code block. Fixed.

@yuyichao
Copy link
Contributor

yuyichao commented Sep 1, 2015

"Package Development" and "Memory allocation analysis" have their own link in the index page but are only sub pages. All other link in the index page have their own pages. Not sure if this inconsistency matters.

jakebolewski added a commit that referenced this pull request Sep 4, 2015
@jakebolewski jakebolewski merged commit b122bc7 into master Sep 4, 2015
@jakebolewski jakebolewski deleted the jcb/docfixes branch September 4, 2015 20:30
@quinnj
Copy link
Member

quinnj commented Sep 4, 2015

let's celebrate y'all

@nolta
Copy link
Member

nolta commented Sep 4, 2015

🍸 🍸 🍸

@timholy
Copy link
Member

timholy commented Sep 4, 2015

Our heroes!

@bicycle1885
Copy link
Member

🍻 🍶 🍷

@ScottPJones
Copy link
Contributor

Cheers to all the great work done here by some pretty dedicated people! 👏👏👏

@Sisyphuss
Copy link

Cheers!

@KristofferC
Copy link
Member

fig_022

@ScottPJones
Copy link
Contributor

OK, was this the last thing holding up branching a 0.4 pre-release?
(please don't call it RC, unless it really is a candidate [with no changes] for the first release)
If so, I'm going to get sloppily drunk celebrating!

@jakebolewski
Copy link
Member

Check your milestones before you take to the bottle.

@tkelman
Copy link
Contributor

tkelman commented Sep 4, 2015

I'll open a separate issue for the type summary in repl help.

@ScottPJones
Copy link
Contributor

@jakebolewski #11536 (comment) the roadmap for 0.4 was closed, and no other was reopened, so, how do I check the milestones???

@ScottPJones
Copy link
Contributor

Thanks! I hadn't seen that feature of GitHub before, had just been following the 0.4 roadmap issue, but now I'm going to start drinking, after seeing how many things still to be done, most dating from 1-3 years ago!

@ViralBShah
Copy link
Member

👏

@StephenVavasis
Copy link
Contributor

I hope I am posting this in the right place: if you are still looking for documentation issues, including the library documentation, then I would like to bring up an issue related to:

JuliaCollections/DataStructures.jl#114

and the work of @mbauman. The documentation states that an IntSet holds 'nonnegative' integers, and yet trying to insert 0 into an IntSet causes a deprecation warning in all recent masters.

My own preference is to remove the deprecation warning (see the discussion mentioned above), but if the deprecation warning is not removed, then the documentation should be fixed.

MichaelHatherly added a commit to MichaelHatherly/julia that referenced this pull request Sep 27, 2015
Combines the `doc(::Function)` and `doc(::DataType)` methods since they
were nearly identical.

Enables searching for docstrings attached to specific constructors in the
same manner as is currently allowed for normal methods. Prior to this it
resulted in a no method error.

Fixes JuliaLang#12701 where splatted method docs were shadowing others. Although not
the original cause of the issue, with the change to storing signatures
as `Tuple{...}` (in JuliaLang#12835) this meant that `f(x...)` was stored as
`Tuple{Vararg{Any}}` which is equal to `Tuple`. Since the default value of
`sig` was `Tuple` not all matching docs were returned in the example
provided in JuliaLang#12701.
tkelman pushed a commit that referenced this pull request Sep 28, 2015
Combines the `doc(::Function)` and `doc(::DataType)` methods since they
were nearly identical.

Enables searching for docstrings attached to specific constructors in the
same manner as is currently allowed for normal methods. Prior to this it
resulted in a no method error.

Fixes #12701 where splatted method docs were shadowing others. Although not
the original cause of the issue, with the change to storing signatures
as `Tuple{...}` (in #12835) this meant that `f(x...)` was stored as
`Tuple{Vararg{Any}}` which is equal to `Tuple`. Since the default value of
`sig` was `Tuple` not all matching docs were returned in the example
provided in #12701.

(cherry picked from commit 1b1e6c6)
ref #13330
skumagai pushed a commit to skumagai/julia that referenced this pull request Oct 9, 2015
Combines the `doc(::Function)` and `doc(::DataType)` methods since they
were nearly identical.

Enables searching for docstrings attached to specific constructors in the
same manner as is currently allowed for normal methods. Prior to this it
resulted in a no method error.

Fixes JuliaLang#12701 where splatted method docs were shadowing others. Although not
the original cause of the issue, with the change to storing signatures
as `Tuple{...}` (in JuliaLang#12835) this meant that `f(x...)` was stored as
`Tuple{Vararg{Any}}` which is equal to `Tuple`. Since the default value of
`sig` was `Tuple` not all matching docs were returned in the example
provided in JuliaLang#12701.
@coveralls
Copy link

Coverage Status

Coverage decreased (-27.6%) to 51.231% when pulling 5b8cb58 on jcb/docfixes into 5cc32bb on master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Indicates an unexpected problem or unintended behavior docs This change adds or pertains to documentation docsystem The documentation building system priority This should be addressed urgently
Projects
None yet
Development

Successfully merging this pull request may close these issues.