Skip to content
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

Make DataFrame.to_html output full content #24841

Merged
merged 15 commits into from
Mar 3, 2019
Merged
Show file tree
Hide file tree
Changes from 11 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion doc/source/whatsnew/v0.24.1.rst
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ Bug Fixes

**I/O**

-
- Bug in :func:`DataFrame.to_html()` where values were truncated using display options instead of outputting the full content (:issue:`17004`)
-
-

Expand Down
13 changes: 11 additions & 2 deletions pandas/io/formats/html.py
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@

from pandas.core.dtypes.generic import ABCMultiIndex

from pandas import compat
from pandas import compat, option_context
import pandas.core.common as com
from pandas.core.config import get_option

Expand Down Expand Up @@ -320,9 +320,15 @@ def _write_header(self, indent):

self.write('</thead>', indent)

def _get_formatted_values(self):
jreback marked this conversation as resolved.
Show resolved Hide resolved
with option_context('display.max_colwidth', 999999):
WillAyd marked this conversation as resolved.
Show resolved Hide resolved
fmt_values = {i: self.fmt._format_col(i)
for i in range(self.ncols)}
return fmt_values

def _write_body(self, indent):
self.write('<tbody>', indent)
fmt_values = {i: self.fmt._format_col(i) for i in range(self.ncols)}
fmt_values = self._get_formatted_values()

# write values
if self.fmt.index and isinstance(self.frame.index, ABCMultiIndex):
Expand Down Expand Up @@ -486,6 +492,9 @@ class NotebookFormatter(HTMLFormatter):
DataFrame._repr_html_() and DataFrame.to_html(notebook=True)
"""

def _get_formatted_values(self):
return {i: self.fmt._format_col(i) for i in range(self.ncols)}

def write_style(self):
# We use the "scoped" attribute here so that the desired
# style properties for the data frame are not then applied
Expand Down
23 changes: 23 additions & 0 deletions pandas/tests/io/formats/test_to_html.py
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,15 @@

import pandas.io.formats.format as fmt

lorem_ipsum = (
"Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod"
" tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim"
" veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex"
" ea commodo consequat. Duis aute irure dolor in reprehenderit in"
" voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur"
" sint occaecat cupidatat non proident, sunt in culpa qui officia"
" deserunt mollit anim id est laborum.")


def expected_html(datapath, name):
"""
Expand Down Expand Up @@ -600,3 +609,17 @@ def test_to_html_render_links(render_links, expected, datapath):
result = df.to_html(render_links=render_links)
expected = expected_html(datapath, expected)
assert result == expected


@pytest.mark.parametrize('method,expected', [
('to_html', lambda x:lorem_ipsum),
('_repr_html_', lambda x:lorem_ipsum[:x - 4] + '...') # regression case
])
@pytest.mark.parametrize('max_colwidth', [10, 20, 50, 100])
def test_ignore_display_max_colwidth(method, expected, max_colwidth):
# see gh-17004
df = DataFrame([lorem_ipsum])
with pd.option_context('display.max_colwidth', max_colwidth):
result = getattr(df, method)()
expected = expected(max_colwidth)
assert expected in result