Skip to content
This repository has been archived by the owner on Nov 25, 2024. It is now read-only.

Commit

Permalink
Fix pip dependencies (#118)
Browse files Browse the repository at this point in the history
Move conda-only dependencies out of `pyproject` and `requirements` sections in `dependencies.yaml`.

Authors:
  - Paul Taylor (https://github.com/trxcllnt)

Approvers:
  - Bradley Dice (https://github.com/bdice)
  - Brad Rees (https://github.com/BradReesWork)

URL: #118
  • Loading branch information
trxcllnt authored Jan 22, 2024
1 parent c592185 commit 8cb7c5c
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 5 deletions.
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -87,3 +87,4 @@ cpp/.idea/
cpp/cmake-build-debug/
pylibwholegraph/.idea/
pylibwholegraph/cmake-build-debug/
compile_commands.json
16 changes: 11 additions & 5 deletions dependencies.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -192,21 +192,23 @@ dependencies:
packages: []
test_cpp:
common:
- output_types: [conda, requirements]
- output_types: [conda]
packages:
- nccl
test_python:
common:
- output_types: [conda, requirements]
- output_types: [conda]
packages:
- c-compiler
- cxx-compiler
- nccl
- output_types: [conda, requirements]
packages:
- ninja
- numpy>=1.17
- pytest
- pytest-forked
- pytest-xdist
- nccl
specific:
- output_types: [conda, requirements]
matrices:
Expand Down Expand Up @@ -273,10 +275,12 @@ dependencies:
packages:
docs:
common:
- output_types: [conda]
packages:
- *doxygen
- output_types: [conda, requirements]
packages:
- breathe
- *doxygen
- graphviz
- ipython
- ipykernel
Expand All @@ -297,10 +301,12 @@ dependencies:
clang_tools:
common:
- output_types: [conda, requirements]
packages:
- gitpython
- output_types: conda
packages:
- clangxx==16.0.6
- clang-tools==16.0.6
- gitpython
python_build_wheel:
common:
- output_types: [pyproject]
Expand Down

0 comments on commit 8cb7c5c

Please sign in to comment.