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

[Docs] Readthedocs fix #70

Merged
merged 2 commits into from
Oct 24, 2023
Merged

[Docs] Readthedocs fix #70

merged 2 commits into from
Oct 24, 2023

Conversation

skim0119
Copy link
Contributor

I fixed the RTD; the build is completed. The issue was the recent change in their policy (page), I had to remove system_packages: true and install dependency packages using poetry. There are some other warnings because we moved around the modules, but we can fix them later.

I also changed one minor import numpy.core.umath_tests.inner1d as it is deprecated, and it was one of the reason why documentation build was failing.

@skim0119 skim0119 added the documentation Improvements or additions to documentation label Oct 24, 2023
@skim0119 skim0119 requested a review from frthjf October 24, 2023 17:32
@skim0119 skim0119 self-assigned this Oct 24, 2023
Copy link
Collaborator

@frthjf frthjf left a comment

Choose a reason for hiding this comment

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

Thank you!

@frthjf frthjf merged commit 27a76a2 into main Oct 24, 2023
9 checks passed
@frthjf frthjf deleted the wip/rtd_build branch October 24, 2023 17:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants