Skip to content

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

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

jorisvandenbossche
Copy link
Member

@jorisvandenbossche jorisvandenbossche commented Jul 15, 2025

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.

@@ -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")
Copy link
Member Author

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
Copy link
Member Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant