-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
DOC: FutureWarning in Sphinx build when calling read_parquet #32833
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
Conversation
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.
lgtm, thanks for the fix @MomIsBestFriend
@datapythonista Is there a reason to limit the line characters length to 80, in I don't mean disable the check, but 88 characters seems to me more reasonable rather than 80. |
Even if |
doc/source/user_guide/scale.rst
Outdated
ddf = dd.read_parquet("data/timeseries/ts*.parquet", engine="pyarrow") | ||
ddf = dd.read_parquet( | ||
"data/timeseries/ts*.parquet", engine="pyarrow", auto_mkdir=True | ||
) |
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.
Another option is to set :okwarning:
for now, as I suppose this is something to be fixed on the dask side
@MomIsBestFriend can you just merge master? I just merged another PR that fixes the theme name error you now get |
CI is green :) |
Thanks for fixing this @MomIsBestFriend |
… calling read_parquet
…ead_parquet (#32847) Co-authored-by: MomIsBestFriend <[email protected]>
black pandas
git diff upstream/master -u -- "*.py" | flake8 --diff