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

Link wholegrah and cugraphops XML docs #3906

Merged
merged 6 commits into from
Oct 11, 2023

Conversation

AyodeAwe
Copy link
Contributor

@AyodeAwe AyodeAwe commented Oct 2, 2023

Configures breathe and links the uploaded wholegraph and cugraph-ops XML docs artifacts.

@AyodeAwe AyodeAwe added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Oct 2, 2023
@BradReesWork BradReesWork added this to the 23.12 milestone Oct 2, 2023
@AyodeAwe AyodeAwe marked this pull request as ready for review October 3, 2023 17:44
@AyodeAwe AyodeAwe requested review from a team as code owners October 3, 2023 17:44
ci/build_docs.sh Outdated Show resolved Hide resolved
docs/cugraph/source/conf.py Outdated Show resolved Hide resolved
@AyodeAwe
Copy link
Contributor Author

AyodeAwe commented Oct 3, 2023

/merge

ci/build_docs.sh Outdated Show resolved Hide resolved
@trxcllnt
Copy link
Collaborator

trxcllnt commented Oct 5, 2023

The devcontainers failure is due to rmm==23.12.* and pylibraft==23.12.* are included in a requirements.txt section. Those packages aren't published to our package index, the -cu{11,12} suffixed ones are. The fixes are in this version of dependencies.yaml.

@AyodeAwe
Copy link
Contributor Author

AyodeAwe commented Oct 10, 2023

The devcontainers failure is due to rmm==23.12.* and pylibraft==23.12.* are included in a requirements.txt section. Those packages aren't published to our package index, the -cu{11,12} suffixed ones are. The fixes are in this version of dependencies.yaml.

@trxcllnt The fixes have been merged in but the error (No matching distribution found for cudf-cu12==23.12.*) seems to persist.

The same error occurs in this run but the run passes.

@AyodeAwe
Copy link
Contributor Author

AyodeAwe commented Oct 10, 2023

In the same logs I see:

Looking in indexes: https://pypi.org/simple, ***pypi.k8s.rapids.ai/simple, https://pypi.nvidia.com/
  Collecting cudf-cu12==23.12.*
    Downloading https://pypi.k8s.rapids.ai/api/package/cudf-cu12/cudf_cu12-23.12.0.231010-cp310-cp310-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (506.3 MB)
       ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 506.3/506.3 MB 4.6 MB/s eta 0:00:00

suggesting that the actual problem might not be a missing dependency, especially as the run continues, although it abruptly stops just ahead while processing the cpp stage:

  [2/532] /usr/bin/sccache /usr/local/cuda/bin/nvcc -forward-unknown-to-host-compiler -ccbin=/usr/bin/g++ -DCUDA_API_PER_THREAD_DEFAULT_STREAM -DCUTLASS_NAMESPACE=raft_cutlass -DFMT_HEADER_ONLY=1 -DRAFT_COMPILED -DRAFT_SYSTEM_LITTLE_ENDIAN=1 -DSPDLOG_FMT_EXTERNAL -DTHRUST_DEVICE_SYSTEM=THRUST_DEVICE_SYSTEM_CUDA -DTHRUST_HOST_SYSTEM=THRUST_HOST_SYSTEM_CPP -Dcugraph_EXPORTS -I/home/coder/cugraph/cpp/../thirdparty -I/home/coder/cugraph/cpp/src -I/home/coder/cugraph/cpp/include -I/home/coder/cugraph/cpp/build/release/_deps/rmm-src/include -I/home/coder/cugraph/cpp/build/release/_deps/thrust-src -I/home/coder/cugraph/cpp/build/release/_deps/thrust-src/dependencies/cub -I/home/coder/cugraph/cpp/build/release/_deps/fmt-src/include -I/home/coder/cugraph/cpp/build/release/_deps/spdlog-src/include -I/home/coder/cugraph/cpp/build/release/_deps/cugraph-ops-src/cpp/include -I/home/coder/cugraph/cpp/build/release/_deps/raft-src/cpp/include -I/home/coder/cugraph/cpp/build/release/_deps/cuco-src/include -I/home/coder/cugraph/c

seems like an OOM issue

@AyodeAwe
Copy link
Contributor Author

Issue seems fixed after running a local build (new sccache caches get uploaded as a side effect and these potentially mean less resource demands for both CI and local subsequent builds)

https://github.com/rapidsai/cugraph/actions/runs/6465954650/job/17580751816?pr=3906

@AyodeAwe
Copy link
Contributor Author

/merge

@rapids-bot rapids-bot bot merged commit 81e33bd into rapidsai:branch-23.12 Oct 11, 2023
76 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Improvement / enhancement to an existing function non-breaking Non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants