suggestion to add /ok to build docs #25
Comments
rapids-bot bot
pushed a commit
to rapidsai/cugraph
that referenced
this issue
May 21, 2024
Closes #4285 I'll report back if it get to render locally. Ran out of mem building the library on my ec2 machine (g5.2xlarge: 32 Gb RAM) but I did just build cudf. I'll try again soon. ~~Also brainstorming here and I probably should upstream this to the rapids CI process. Would be nice to have /ok to test docs which just builds the docs for this PR~~ created nv-gha-runners/nvidia-runners#25 I tried building the docs locally and got ``` WARNING: [autosummary] failed to import cugraph.jaccard_w. Possible hints: * AttributeError: module 'cugraph' has no attribute 'jaccard_w' * ImportError: * ModuleNotFoundError: No module named 'cugraph.jaccard_w' WARNING: [autosummary] failed to import cugraph.overlap_w. Possible hints: * ModuleNotFoundError: No module named 'cugraph.overlap_w' * ImportError: * AttributeError: module 'cugraph' has no attribute 'overlap_w' WARNING: [autosummary] failed to import cugraph.sorensen_w. Possible hints: * ModuleNotFoundError: No module named 'cugraph.sorensen_w' * ImportError: * AttributeError: module 'cugraph' has no attribute 'sorensen_w' ``` Think this comes from https://github.com/rapidsai/cugraph/blob/abe69c0419b67b567d3c8fce91ee1a062d53e385/docs/cugraph/source/api_docs/cugraph/link_prediction.rst#L14 but I may have messed up my build Authors: - Ray Bell (https://github.com/raybellwaves) - Rick Ratzel (https://github.com/rlratzel) Approvers: - Rick Ratzel (https://github.com/rlratzel) URL: #4323
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I apologize if this is the wrong place to make a suggestion. Happy to close this and open it elsewhere.
I've recently made PR's to some of the library where I only updated docs e.g. rapidsai/cugraph#4323
Sometimes if there are only changes to a docstring it may be worth first only testing the docs build e.g. with a command like
/ok to build docswhich runs just thedocs-build / buildaction (which I think also offers a preview to the docs). I understand the need to finally do a full CI to check that nothing is broken so is the change renders as expected you could like run/ok to test.The text was updated successfully, but these errors were encountered: