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

[BUG]: cuGraph does not build from source. #2892

Closed
2 tasks done
ccoulombe opened this issue Nov 6, 2022 · 3 comments · Fixed by #2904
Closed
2 tasks done

[BUG]: cuGraph does not build from source. #2892

ccoulombe opened this issue Nov 6, 2022 · 3 comments · Fixed by #2904
Labels
bug Something isn't working

Comments

@ccoulombe
Copy link

ccoulombe commented Nov 6, 2022

Version

22.12

Which installation method(s) does this occur on?

From source, not conda

Describe the bug.

cuGraph does not build from source with bash build.sh.

  Could not find a package configuration file provided by "cugraph-ops"
  (requested version 22.12.00) with any of the following names:

    cugraph-opsConfig.cmake
    cugraph-ops-config.cmake

It is missing cugraph-ops and conda is not an option.

Both #2280 and #2231 had the same issue, but it is still an issue.

Minimum reproducible example

git clone https://github.com/rapidsai/cugraph.git
cd cugraph
bash build.sh

Relevant log output

Could not find a package configuration file provided by "cugraph-ops"
  (requested version 22.12.00) with any of the following names:

    cugraph-opsConfig.cmake
    cugraph-ops-config.cmake

Environment details

No response

Other/Misc.

Where can one find cugraph-ops? If it is a closed source, can cugraph-ops be set to off by default instead?

Code of Conduct

  • I agree to follow cuGraph's Code of Conduct
  • I have searched the open bugs and have found no duplicates for this bug report
@ccoulombe ccoulombe added ? - Needs Triage Need team to review and classify bug Something isn't working labels Nov 6, 2022
@BradReesWork BradReesWork removed the ? - Needs Triage Need team to review and classify label Nov 8, 2022
@rlratzel
Copy link
Contributor

rlratzel commented Nov 9, 2022

Hi @ccoulombe , cugraph-ops is an optional closed-source package that's used by certain sampling algorithms. It can be installed using conda, but if conda is not an option then the binaries will need to be downloaded and installed another way (I think we have support for pip for that package, but I would need to get you the details).

If you don't need support for every sampling algorithm, then you can still build cugraph without it. If you add -DUSE_CUGRAPH_OPS=OFF to the cmake call in the build.sh script here, you'll be able to build and run cugraph without cugraph-ops.

I've also created an issue to address this problem which you can track here.

@ccoulombe
Copy link
Author

@rlratzel Thanks!

I did search for the library, on PyPI and elsewhere, without luck.

Indeed, that's what I ended up doing : compiling without cugraph-ops by patching the build script.
I could do a PR if you are not already working on it.

Cheers!

@rlratzel
Copy link
Contributor

rlratzel commented Nov 9, 2022

Glad to hear you got it working @ccoulombe . Thanks for offering to submit a PR, but I had one started and just submitted it here, which should auto-close this issue when merged.

rapids-bot bot pushed a commit that referenced this issue Nov 15, 2022
)

closes #2902 
closes #2892 

Allows from-source users to build without `cugraphops`. This will disable the sampling algos that depend on cugraphops, but otherwise cugraph builds normally.

Tested by removing `libcugraphops` from my environment, ensured the build error described in #2892 was seen, built again with the new option and observed the build succeed.

Authors:
  - Rick Ratzel (https://github.com/rlratzel)
  - Chuck Hastings (https://github.com/ChuckHastings)

Approvers:
  - Chuck Hastings (https://github.com/ChuckHastings)

URL: #2904
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants