-
-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Add omniscidb-cuda #9225
Add omniscidb-cuda #9225
Conversation
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
@xmnlab @andrewseidl please confirm that you are willing to be the maintainers of omnscidb-cuda conda package. |
Confirmed. |
confirmed! thanks @pearu! |
Hi! This is the friendly automated conda-forge-linting service. I wanted to let you know that I linted all conda-recipes in your PR ( Here's what I've got... For recipes/omniscidb-cuda/recipe:
|
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
This PR is blocked by cudatoolkit-dev issues. The background of the linux_64 check failure is described here: conda-forge/cudatoolkit-dev-feedstock#16 (comment) |
Yeah, I think that cudatoolkit-dev is not a good solution for building packages. @jakirkham - what is the status on being able to use a cuda-enable docker container? |
The blocking issue is eliminated (not using |
There's a lot of files installed that could potentially conflict with other packages.
|
This will likely require some planning as the project's cmake setup currently assumes it's being installed to a dedicated dir, eg Is there any precedent for other packages to be installed like that, or must everything go to the general |
You could install everything into |
Omniscidb is now installed to The activation script prepends |
Co-Authored-By: Isuru Fernando <isuruf@gmail.com>
Checklist
url
) rather than a repo (e.g.git_url
) is used in yourrecipe (see
here
for more details)