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.

Sign up for GitHub

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

Jump to bottom

Add changelog for dataframe removal #11654

Merged
merged 3 commits into from
Jan 16, 2025
Merged

Add changelog for dataframe removal #11654

merged 3 commits into from
Jan 16, 2025

Conversation

phofl
Copy link
Collaborator

@phofl phofl commented Jan 9, 2025

  • Closes #xxxx
  • Tests added / passed
  • Passes pre-commit run --all-files

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jan 9, 2025 •
edited
Loading

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   4h 23m 30s ⏱️ - 4m 49s
 17 156 tests + 5   15 961 ✅ + 5   1 195 💤 ±0  0 ❌ ±0 
211 366 runs  +62  194 190 ✅ +61  17 176 💤 +1  0 ❌ ±0 

Results for commit d8fa49e. ± Comparison against base commit ae3fc2e.

This pull request removes 230 and adds 235 tests. Note that renamed tests count towards both. 
dask.dataframe.dask_expr.tests.test_resample ‑ test_resample_agg
dask.dataframe.dask_expr.tests.test_resample ‑ test_resample_apis[count-kwargs0]
dask.dataframe.dask_expr.tests.test_resample ‑ test_resample_apis[count-kwargs1]
dask.dataframe.dask_expr.tests.test_resample ‑ test_resample_apis[first-kwargs0]
dask.dataframe.dask_expr.tests.test_resample ‑ test_resample_apis[first-kwargs1]
dask.dataframe.dask_expr.tests.test_resample ‑ test_resample_apis[last-kwargs0]
dask.dataframe.dask_expr.tests.test_resample ‑ test_resample_apis[last-kwargs1]
dask.dataframe.dask_expr.tests.test_resample ‑ test_resample_apis[max-kwargs0]
dask.dataframe.dask_expr.tests.test_resample ‑ test_resample_apis[max-kwargs1]
dask.dataframe.dask_expr.tests.test_resample ‑ test_resample_apis[mean-kwargs0]
…

dask.array.tests.test_creation ‑ test_from_array_getitem_fused
dask.array.tests.test_slicing ‑ test_vindex_with_dask_array
dask.dataframe.dask_expr.io.tests.test_io ‑ test_from_dask_array_projection
dask.dataframe.dask_expr.tests.test_merge_asof ‑ test_merge_asof_indexed_optimiser
dask.dataframe.io.tests.test_parquet ‑ test_filter_with_struct_column
dask.dataframe.tseries.tests.test_resample_expr ‑ test_resample_agg
dask.dataframe.tseries.tests.test_resample_expr ‑ test_resample_apis[count-kwargs0]
dask.dataframe.tseries.tests.test_resample_expr ‑ test_resample_apis[count-kwargs1]
dask.dataframe.tseries.tests.test_resample_expr ‑ test_resample_apis[first-kwargs0]
dask.dataframe.tseries.tests.test_resample_expr ‑ test_resample_apis[first-kwargs1]
…

♻️ This comment has been updated with latest results.

@fjetter fjetter mentioned this pull request Jan 10, 2025
Closed
8 tasks
hendrikmakait
hendrikmakait reviewed Jan 10, 2025
View reviewed changes
docs/source/changelog.rst Outdated Show resolved Hide resolved
docs/source/changelog.rst Outdated Show resolved Hide resolved
@fjetter fjetter merged commit 116702d into dask:main Jan 16, 2025
25 of 26 checks passed
@phofl phofl deleted the dataframe-changelog branch January 16, 2025 12:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hendrikmakait hendrikmakait hendrikmakait left review comments

@fjetter fjetter fjetter approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@phofl @hendrikmakait @fjetter