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

Clean up environment file to mitigate sphinx version problem #542

Merged
merged 4 commits into from
Sep 27, 2023

Conversation

kafitzgerald
Copy link
Collaborator

Clean up environment file to mitigate sphinx version problem:

  • Removes old sphinx theme from the environment file and related template file
  • Removes jupyter and sphinx from the environment file so they're just installed as dependencies (hopefully this mitigates the sphinx version issue we are having)

Closes #541.

@kafitzgerald kafitzgerald marked this pull request as draft September 27, 2023 17:51
@kafitzgerald
Copy link
Collaborator Author

Ok, so now this is just failing because of the matplotlib update as logged in NCAR/geocat-viz#160.

@kafitzgerald kafitzgerald marked this pull request as ready for review September 27, 2023 18:55
@kafitzgerald
Copy link
Collaborator Author

Unfortunately, the ReadTheDocs build is just going to be failing for a bit here. I did verify that it's back to the "Command killed due to timeout or excessive memory consumption" mode of failure.

My take is that we should start merging some things as long as we're confident that everything else is passing and the RTD failure is for the known reason and I can see if there's another potential interim solution (given that we are keeping the notebooks for a bit per #524 and current timelines).

@jukent jukent merged commit 3b29885 into NCAR:main Sep 27, 2023
6 of 7 checks passed
@kafitzgerald kafitzgerald deleted the issue541 branch November 20, 2023 18:00
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.

sphinx-book-theme is failing with sphinx 7
2 participants