Skip to content

ENH: improve optional import error message #61597

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
Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

ENH: improve optional import error message #61597

wants to merge 6 commits into from
+10 −10

Conversation

KevsterAmp
Copy link
Contributor

@KevsterAmp KevsterAmp commented Jun 7, 2025 •
edited
Loading

@KevsterAmp KevsterAmp marked this pull request as draft June 7, 2025 14:29
@KevsterAmp KevsterAmp marked this pull request as ready for review June 12, 2025 17:55
@KevsterAmp
Copy link
Contributor Author

I'm not sure how to fix the CI errors:

Error: Failed to CreateArtifact: Received non-retryable error: Failed request: (409) Conflict: an artifact with this name already exists on the workflow run
@yuanx749
Copy link
Contributor

One change is missing:

pandas/pandas/tests/io/test_parquet.py

Line 337 in 675c81c

match = "Missing optional dependency .fastparquet."

I expect the CI will pass once that’s fixed.
@KevsterAmp
Copy link
Contributor Author

thanks @yuanx749, confirmed on the previous CI errors that the fastparquet was the issue
@simonjayhawkins simonjayhawkins added the Error Reporting Incorrect or improved errors from pandas label Jun 25, 2025
@KevsterAmp
Copy link
Contributor Author

@rhshadrach - can you kindly review this? thanks!
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Error Reporting Incorrect or improved errors from pandas
3 participants
@KevsterAmp @yuanx749 @simonjayhawkins