-
Notifications
You must be signed in to change notification settings - Fork 70
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
Docs are not building #382
Comments
@andrewtavis shall I? |
Would be great, @Khushalsarode :) |
CC also @AnishereMariam here, as your changes to the docs likely won't be reflected until this is figured out :) |
Hey! I'd love to help with this issue in any way I can. Let me know how I can assist! |
I have been trying to debug this all night. I keep hitting roadblocks. @Khushalsarode, can we please have a discussion session on Matrix regarding this issue? I'd like us to fix it soon. @OmarAI2003, your input is highly welcomed. We can all discuss on Matrix if that is fine with you. |
Let me know if you all need support! |
Thanks for the update! I sent you a message on Matrix, so we can discuss it there when you have a moment. Looking forward to collaborating! |
Hi @andrewtavis,Hi everyone, I wanted to update you on the progress with the Sphinx documentation build . After making adjustments to the
I'm want to keep working on this, so any suggestions or insights would be greatly appreciated! Thanks! |
Thanks for the efforts here, @OmarAI2003! Re import errors, please replace the words to replace and words to ignore with empty list declarations, so Re docstring error, yes we need to have all the docstrings with the same format so they're parsed correctly by auto documentation, so they all need to be numpy docstring form. I think I fixed the format, so it's a section. We have "Raises" in many docstrings, so it's likely "Example" that's problematic. Can you put the example in the description in the top and see if it works? I'm not sure right now on the other two, but hopefully the docs will build from there and we can open new issues for those? Thanks and let us know how it's going 😊 |
Another update on the documentation build process. I've addressed the import errors by replacing the references to the removed functions with empty list declarations, as you suggested. Regarding the other two issues you mentioned @andrewtavis , since I've successfully built the documentation without any errors or warnings, it seems we may not need to open new issues at this time. |
@andrewtavis Documentation Running without error now Firstly when I tried to run the documentation using make html file:
There was some package issue that docutils was not compatible with m2r2
then read about: changes made in config.pyBefore:
After:
Tried by autobuild and make.bat html autobuild:
|
@OmarAI2003, does the above also get solved by your PR? |
ya! saw his PR solving issue, that why I only just shared the description to issue. |
The work in my PR was based solely on the modules listed in the |
yes! @OmarAI2003 |
Thanks, @Khushalsarode! This is indeed a correct and valid edit for autobuild, and I've included it in a later commit. |
Docs are still failing, @OmarAI2003:
Command for this is: python -m sphinx -T -b html -d _build/doctrees -D language=en . $READTHEDOCS_OUTPUT/html |
Yeah @andrewtavis , this is because the edits Kushal suggested require the recommonmark module to be installed in order to build using sphinx-autobuild. If autobuild isn't necessary, we can remove these edits or otherwise install recommonmark. The (Bash) command you suggested: |
I just copied your command and pasted it and it worked as shown in the image. What do you recommend—removing the edits or installing the recommonmark module? |
Let's remove the edits so that we can keep it as simple as possible :) If we don't need a dependency, let's not include it. Can you send along a PR? |
Alright I will |
Thanks for the great work here, @OmarAI2003, and also for the conversation and collaboration in fixing everything, all! @mhmohona: Let's keep in mind that the docs are slowly becoming wayyyy out of date with all of the work. Maybe you can do a scan and make some good first issues to fix things in them? |
Others are welcome to check the docs and open issues as well! |
Hi Andrew, I reached out to @mhmohona and had a brief chat about the documentation. I've also submitted this PR #465 |
Terms
Behavior
Something that I've noticed recently is that the docs have stopped building successfully. The original error message was:
Looking into it,
reprunicode
was removed from Sphinx, one of the dependencies, and an initial attempt to fix this was to update the dependencies withsphinx-rtd-theme>=3.0.0
. The new error is:Suggestions on this and a PR would be welcome :)
The text was updated successfully, but these errors were encountered: