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

Slight update to dask-expr warning #10916

Merged
merged 1 commit into from
Feb 9, 2024

Conversation

jrbourbeau
Copy link
Member

This PR is a small follow-up to #10912 that:

  • Fully spells out the config setting. Importantly, it needs to happen before dask.dataframe is imported.
  • Point to the issue tracker as a place to give feedback (I think that's the main point of issuing this warning in the first place)
  • Has some minor wording changes

cc @phofl @fjetter

Copy link
Contributor

github-actions bot commented Feb 9, 2024

Unit Test Results

See test report for an extended history of previous test failures. This is useful for diagnosing flaky tests.

     15 files  ±0       15 suites  ±0   3h 18m 47s ⏱️ - 1m 35s
 12 988 tests ±0   12 059 ✅ +1     929 💤 ±0  0 ❌  - 1 
160 507 runs  ±0  144 031 ✅  - 3  16 476 💤 +4  0 ❌  - 1 

Results for commit d6c9877. ± Comparison against base commit 3e775b4.

@jrbourbeau jrbourbeau merged commit 20b6e2d into dask:main Feb 9, 2024
28 of 29 checks passed
@jrbourbeau jrbourbeau deleted the dask-expr-warning branch February 9, 2024 16:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants