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

Explain that matplotlib must be built before the HTML documentation #1660

Merged
merged 4 commits into from Jan 23, 2013
Merged

Explain that matplotlib must be built before the HTML documentation #1660

merged 4 commits into from Jan 23, 2013

Conversation

vterron
Copy link

@vterron vterron commented Jan 14, 2013

Attempting to generate the HTML documentation right after cloning the
repository results in a cryptic error message as one of the required
files, mpl-data/matplotlibrc, is created when matplotlib is built. It
seems that other people have come across this problem in the past [1],
so mentioning it here might be of help to newcomers.

[1] http://sourceforge.net/mailarchive/message.php?msg_id=21478615

Attempting to generate the HTML documentation right after cloning the
repository results in a cryptic error message as one of the required
files, mpl-data/matplotlibrc, is created when matplotlib is built. It
seems that other people have come across this problem in the past [1],
so mentioning it here might be of help to newcomers.

[1] http://sourceforge.net/mailarchive/message.php?msg_id=21478615
@mdboom
Copy link
Member

mdboom commented Jan 14, 2013

It's actually more than that -- matplotlib must be installed (so that the docstrings are accessible) before the documentation can be built.

@vterron
Copy link
Author

vterron commented Jan 14, 2013

Should I add a second commit and mention that in doc/README.txt too?

@NelleV
Copy link
Member

NelleV commented Jan 15, 2013

This is something general to python projects and sphinx, not something specific to matplotlib. Sphinx uses the installed version of the documentation to built the documentation of this project. I don't think it is worth explaining the matplotlibrc creation in the README. I'd just mention that the package needs to be installed before building the documentation.

@pelson
Copy link
Member

pelson commented Jan 15, 2013

This is something general to python projects and sphinx, not something specific to matplotlib. Sphinx uses the installed version of the documentation to built the documentation of this project. I don't think it is worth explaining the matplotlibrc creation in the README. I'd just mention that the package needs to be installed before building the documentation.

@NelleV - your absolutely right - most python packages depend on the package they are documenting being installed before you can build the docs with sphinx. It is worth noting however, that the biggest python project of all, namely python-core, doesn't have this requirement (and instead has a lot of duplication of documentation and a lot of objects which do not have docstrings at all!).

If @vterron (or anybody else) has tripped over this, and we have it in our power to make it easier for everybody to be able to get involved with improving the docs, then I think we should.

@vterron - yes, lets get the message in a state where we are happy for it to get merged, and if, at that point, we have a big commit list, we can always squash it down to a single commit.

Cheers,

@mdboom
Copy link
Member

mdboom commented Jan 15, 2013

Perhaps a better error message from make.py when we don't have a matplotlib installation or a matplotlibrc file would be even better -- we can document all we like, but often a clear message why something is going wrong is even better.

Víctor Terrón added 3 commits January 20, 2013 20:35
Even if the README.txt file explicitly mentions that matplotlib must
have been built before the documentation can be generated, a better
error message will be of help to those developers who happen not to
read it.
@vterron
Copy link
Author

vterron commented Jan 20, 2013

@pelson - the README now explains that matplotlib must have been built and installed before the docs can be generated. As @mdboom suggested, more specific error messages are shown to the user if that is not the case.

@pelson
Copy link
Member

pelson commented Jan 21, 2013

This looks good to me. 👍
@mdboom - please comment & merge when happy.

Thanks @vterron.

@mdboom
Copy link
Member

mdboom commented Jan 23, 2013

Looks good.

mdboom added a commit that referenced this pull request Jan 23, 2013
Explain that matplotlib must be built before the HTML documentation
@mdboom mdboom merged commit ffdd946 into matplotlib:master Jan 23, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants