CornerPlotter broken API fixed for web pages #346
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ons is the problem in particular.
…h py2 and py3 docs.
…s just fix python3 pages then.
Error: Error installing micromamba: Destination file path /home/runner/micromamba-bin/micromamba already exists
|
I already manually ran the documentation building and page deployment from the new branch, so, therefore, the web pages already show now the CornerPlotter API correctly. Only thing that I don't know is why only some functions and their docstrings are shown for that class. But that is maybe determined somewhere. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
As mentioned in issue #306 the CornerPlotter API has not been showing up in the documentation pages for a while. That is now fixed by installing getdist and nestcheck in the GitHub Action that builds the documentation pages. Apparently, those packages were required for CornerPlotter to appear in the documentation.
I only fixed this for the Python3 web pages. For Python2 pages the installation of getdist and nestcheck did not seem to work with same command, and I did not spend time checking alternative ways to do that. Having the full API only in the Python3 pages is probably still ok.