-
Notifications
You must be signed in to change notification settings - Fork 36
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
refactor!: feature flags reorganizization #800
Conversation
nightly
instead of simd
nightly
instead of simd
nightly
instead of simd
@@ -24,6 +24,8 @@ | |||
|
|||
### Other breaking changes | |||
|
|||
- The `simd` feature of the Rust library is removed in favor of |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you add details on whether it changes something for the users? Should they change anything?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since we can update the NEWS file later, could you approve it for now?
It contains changes to the manifest file, and given the caching and the number of jobs triggered, I don't want to prolong this PR with only a documentation update.
@@ -51,6 +53,10 @@ | |||
- `pl$threadpool_size()` is deprecated and will be removed in 0.15.0. Use | |||
`pl$thread_pool_size()` instead (#784). | |||
|
|||
### Other improvements | |||
|
|||
- The `sql` feature is included in the default feature (#800). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same
Close #300
Match with py-polars.
polars/nightly
includespolars/simd
, and py-polars does not specifypolars/simd
directly but includespolras/nightly
in thenightly
feature.default
feature includes thesql
feature (becausesql
no longer requires nightly toolchain).