Skip to content

Make DataFrame.to_html output full content #24841

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 15 commits into from
Mar 3, 2019
Merged
Show file tree
Hide file tree
Changes from 2 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
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):
with option_context('display.max_colwidth', 999999):
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
19 changes: 19 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,8 @@

import pandas.io.formats.format as fmt

lorum_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." # noqa
Copy link
Contributor

Choose a reason for hiding this comment

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

can you be specificerror when using noqa

Copy link
Member Author

Choose a reason for hiding this comment

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

if your happy using that text, i'll break the string up over several lines instead of using the # noqa directive

Copy link
Contributor

Choose a reason for hiding this comment

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

yes that would be better actually

Copy link
Member Author

Choose a reason for hiding this comment

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

or make it a fixture, or use an existing fixture. i'm open to suggestions

Copy link
Contributor

Choose a reason for hiding this comment

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

oh a fixture would be good

Copy link
Member Author

Choose a reason for hiding this comment

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

top level conftest.py? session scope is ok for immutable string



def expected_html(datapath, name):
"""
Expand Down Expand Up @@ -600,3 +602,20 @@ 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('max_colwidth', [10, 20, 50, 100])
def test_display_max_colwidth(max_colwidth):
df = DataFrame([lorum_ipsum])
with pd.option_context('display.max_colwidth', max_colwidth):
result = df._repr_html_()
expected = lorum_ipsum[:max_colwidth - 4] + '...'
assert expected in result


@pytest.mark.parametrize('max_colwidth', [10, 20, 50, 100])
def test_ignore_display_max_colwidth(max_colwidth):
df = DataFrame([lorum_ipsum])
with pd.option_context('display.max_colwidth', max_colwidth):
result = df.to_html()
assert lorum_ipsum in result