-
Notifications
You must be signed in to change notification settings - Fork 933
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
[FEA] Create a single RAPIDS library #1234
Labels
Comments
This is out of scope for now, possibly to be revisited in the future. |
lithomas1
added a commit
that referenced
this issue
Aug 5, 2024
## Description <!-- Provide a standalone description of changes in this PR. --> <!-- Reference any issues closed by this PR with "closes #1234". --> <!-- Note: The pull request title will be included in the CHANGELOG. --> Adapts to IR changes in polars 1.4 and handles nrows/skiprows a little more correctly. ## Checklist - [ ] I am familiar with the [Contributing Guidelines](https://github.com/rapidsai/cudf/blob/HEAD/CONTRIBUTING.md). - [ ] New or existing tests cover these changes. - [ ] The documentation is up to date with these changes. --------- Co-authored-by: Lawrence Mitchell <lmitchell@nvidia.com>
AyodeAwe
pushed a commit
that referenced
this issue
Nov 6, 2024
## Description <!-- Provide a standalone description of changes in this PR. --> <!-- Reference any issues closed by this PR with "closes #1234". --> <!-- Note: The pull request title will be included in the CHANGELOG. --> The case-sensitive name KvikIO is will throw off `find_package` searches, particularly after rapidsai/devcontainers#414 make the usage consistent in devcontainers. ## Checklist - [x] I am familiar with the [Contributing Guidelines](https://github.com/rapidsai/cudf/blob/HEAD/CONTRIBUTING.md). - [x] New or existing tests cover these changes. - [x] The documentation is up to date with these changes.
AyodeAwe
pushed a commit
that referenced
this issue
Feb 27, 2025
## Description <!-- Provide a standalone description of changes in this PR. --> <!-- Reference any issues closed by this PR with "closes #1234". --> <!-- Note: The pull request title will be included in the CHANGELOG. --> Adds an xfail list to the narwhals tests we run using cudf. Note: We can update/replace the dict when running Narwhals with cudf.pandas. xref #18031 ## Checklist - [ ] I am familiar with the [Contributing Guidelines](https://github.com/rapidsai/cudf/blob/HEAD/CONTRIBUTING.md). - [ ] New or existing tests cover these changes. - [ ] The documentation is up to date with these changes. --------- Co-authored-by: Vyas Ramasubramani <vyasr@nvidia.com> Co-authored-by: GALI PREM SAGAR <sagarprem75@gmail.com> Co-authored-by: Bradley Dice <bdice@bradleydice.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Is your feature request related to a problem? Please describe.
Hi all,
I would like to share with you the following idea:
So just as we have the following imports:
why do not create a rapids library that could be imported as follows?
or
I am aware that there are pros about having each library separate, but I think there are also pros about having a single library, such us
Please, share your thoughts about the above.
Thanks!
P.S.: There are also cons about having a huge library, but I think the pros exceed the cons.
The text was updated successfully, but these errors were encountered: