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

Additions to docs/rtd-pip-requirements for readthedocs #58

Merged
merged 1 commit into from
May 20, 2014

Conversation

cdeil
Copy link
Member

@cdeil cdeil commented May 20, 2014

After merging #44 there's now this error on readthedocs:

/var/build/user_builds/photutils/envs/latest/src/astropy/astropy/sphinx/conf.py:156: UserWarning: matplotlib's plot_directive could not be imported. Inline plots will not be included in the output
  "Inline plots will not be included in the output")
WARNING: ConfigurationDefaultMissingWarning: Could not determine version of package photutils Cannot install default profile. If you are importing from source, this is expected. [photutils._astropy_init]
/var/build/user_builds/photutils/checkouts/latest/docs/photutils/findstars.rst:59: ERROR: Unknown directive type "plot".

See full log here: https://readthedocs.org/builds/photutils/1390241/
And the image here doesn't appear.

I think all that's missing is adding matplotlib to the docs/rtd-pip-requirements file, so I'll try that now and also add scipy and scikit-image because the example plotting code runs daofind and will need those.

cc @larrybradley

@cdeil cdeil added this to the 0.1 milestone May 20, 2014
@cdeil cdeil added bug labels May 20, 2014
@cdeil cdeil self-assigned this May 20, 2014
cdeil added a commit that referenced this pull request May 20, 2014
Additions to docs/rtd-pip-requirements for readthedocs
@cdeil cdeil merged commit 2c0f066 into astropy:master May 20, 2014
@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 64ec48e on cdeil:issue_58 into 9769fab on astropy:master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants