Skip to content

it doesn't pull docstrings from python sources #2

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

Closed
certik opened this issue Aug 19, 2010 · 8 comments
Closed

it doesn't pull docstrings from python sources #2

certik opened this issue Aug 19, 2010 · 8 comments

Comments

@certik
Copy link

certik commented Aug 19, 2010

For the sympy documentation, it doesn't seem to be pulling docstrings from the sympy source code:

See e.g. the preview function:

http://readthedocs.org/projects/certik/sympy-docs/docs/modules/printing.html#preview

and the source code, that should pull the docstring:

http://readthedocs.org/projects/certik/sympy-docs/docs/_sources/modules/printing.txt

and here is how it is supposed to look:

http://docs.sympy.org/modules/printing.html#module-sympy.printing.preview

@ericholscher
Copy link
Member

This is due to the fact that we are autogenerating our own conf.py's and disallowing extensions that might execute arbitrary python code. We can whitelist your project and those parts should build correctly.

@certik
Copy link
Author

certik commented Aug 22, 2010

Ah, I see. Do you think you could please white list my project?

@ericholscher
Copy link
Member

Sure, done and rebuilt.

@certik
Copy link
Author

certik commented Aug 22, 2010

Indeed, it now works. Thanks for fixing it --- this was the last big bug, that annoyed me. I think it's now very usable for sympy. Except the issue #1, I guess I shouldn't have deleted the project. Well....

@ericholscher
Copy link
Member

I can remove the old one and put yours in it's spot. Just haven't made that work through the front end yet.

@ericholscher
Copy link
Member

Note that this will change the slug on your urls, but if you haven't used them much, it shouldn't be a problem.

@certik
Copy link
Author

certik commented Aug 22, 2010

Yes, if you could please do that, that'd be absolutely awesome. Thanks again!

@ericholscher
Copy link
Member

Cool. Should be working now. As well as http://sympy.readthedocs.org/ -- cheers.

This issue was closed.
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

No branches or pull requests

2 participants