Skip to content

ENH: max_rows max_cols kw for Styler.to_html + styler.options #42972

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

Merged
merged 25 commits into from
Sep 9, 2021

Conversation

attack68
Copy link
Contributor

@attack68 attack68 commented Aug 10, 2021

one more item towards deprecating DataFrame.to_html for #41693

  • adds options to config
  • update options.rst
  • add tests
  • update to_html docstring
  • whats new
  • update user guide (will be follow on after all options in master tracker added)

and an item towards consistent pandas.options for #41395

Screen Shot 2021-08-29 at 09 26 45

@jreback jreback added Output-Formatting __repr__ of pandas objects, to_string Styler conditional formatting using DataFrame.style labels Aug 11, 2021
# Conflicts:
#	pandas/tests/io/formats/style/test_html.py
@attack68 attack68 changed the title ENH: add kwarg max_rows max_cols to Styler.to_html WIP ENH: add kwarg max_rows max_cols to Styler.to_html Aug 20, 2021
@attack68 attack68 changed the title WIP ENH: add kwarg max_rows max_cols to Styler.to_html ENH: max_rows max_cols kw for Styler.to_html + styler.options Aug 29, 2021
@attack68 attack68 marked this pull request as ready for review August 29, 2021 07:33
@attack68 attack68 added Enhancement IO HTML read_html, to_html, Styler.apply, Styler.applymap labels Aug 29, 2021
@jreback jreback added this to the 1.4 milestone Aug 31, 2021
@@ -836,6 +871,11 @@ def scale_down(rn, cn):
else:
return int(rn * scaling_factor), cn

if max_rows:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this exercised enough in tests?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

added specific cases

Copy link
Contributor

@jreback jreback left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you rebase

# Conflicts:
#	doc/source/user_guide/options.rst
#	doc/source/whatsnew/v1.4.0.rst
#	pandas/core/config_init.py
#	pandas/io/formats/style.py
@jreback
Copy link
Contributor

jreback commented Sep 6, 2021

can you merge master

@attack68 attack68 merged commit 59a06fd into pandas-dev:master Sep 9, 2021
@attack68 attack68 deleted the styler_max_rows_cols branch September 9, 2021 06:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement IO HTML read_html, to_html, Styler.apply, Styler.applymap Output-Formatting __repr__ of pandas objects, to_string Styler conditional formatting using DataFrame.style
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants