Skip to content

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