-
-
Notifications
You must be signed in to change notification settings - Fork 18.7k
DOC: make doc build run with string dtype enabled #61864
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
base: main
Are you sure you want to change the base?
DOC: make doc build run with string dtype enabled #61864
Conversation
cb2f87d
to
85bc3be
Compare
@@ -5281,13 +5281,13 @@ Write to a parquet file. | |||
.. ipython:: python | |||
df.to_parquet("example_pa.parquet", engine="pyarrow") | |||
df.to_parquet("example_fp.parquet", engine="fastparquet") | |||
# df.to_parquet("example_fp.parquet", engine="fastparquet") |
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.
For pyarrow vs fastparquet here, we should maybe not show both constantly in each example? (just show once you can specify the engine, and then in subsequent examples showing the roundtrip just use the default engine?)
@@ -184,7 +184,7 @@ API changes | |||
.. ipython:: python | |||
:okwarning: | |||
dfc.loc[0]['A'] = 1111 | |||
dfc.loc[0]['B'] = 1111 |
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.
The B column are already integers, so this example of chained assignment is still valid this way, but then doesn't include dtype changes
First commit is from #61722 (will be removed here after that PR is merged and this one is rebased), then subsequent commits enable errors on the doc build again and fix issues.