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

⚠️ CI failed ⚠️ - Regression test_csv_basic peak memory #1424

Open
github-actions bot opened this issue Feb 28, 2024 · 4 comments
Open

⚠️ CI failed ⚠️ - Regression test_csv_basic peak memory #1424

github-actions bot opened this issue Feb 28, 2024 · 4 comments
Labels
ci-failure regression

Comments

@github-actions
Copy link

Workflow Run URL
@milesgranger milesgranger changed the title ⚠️ CI failed ⚠️ ⚠️ CI failed ⚠️ - Regression test_csv_basic peak memory Feb 28, 2024
@milesgranger
Copy link
Contributor

Looks like this could be something to keep an eye on:

image

image

static-dashboard.zip

@fjetter
Copy link
Member

fjetter commented Feb 29, 2024

was this before or after enabling dask-expr?

@milesgranger
Copy link
Contributor

I noticed it before merging #1419

@milesgranger
Copy link
Contributor

Checking back in, looks like it's stabilized at the higher avg memory. Should we accept this or look further into it at all?

image

But timing almost looks like a downward trend:

image

static-dashboard.zip

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
ci-failure regression
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@fjetter @milesgranger