-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
ENH: styler.render.repr
option
#43180
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
Changes from 8 commits
f917726
0d459ef
46d933b
0a00907
983d6f4
32343bc
e298e0a
42808a0
64f2b50
5ddf521
a598e54
0951a49
d865a9f
4cae9ac
90f9a17
e2129a1
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -214,6 +214,8 @@ def _repr_html_(self) -> str: | |
""" | ||
Hooks into Jupyter notebook rich display system. | ||
""" | ||
if get_option("styler.render.repr") == "latex": | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. do we actually still need this? or is this something controlled by the notebook itself? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I tried to look this up, and I may be wrong but I think Notebook uses When |
||
return self.to_latex() | ||
return self.to_html() | ||
|
||
def render( | ||
|
@@ -887,7 +889,7 @@ def to_html( | |
``class`` and ``id`` identifiers, or solely the ``<table>`` element without | ||
styling identifiers. | ||
**kwargs | ||
Any additional keyword arguments are passed through to the jinj2 | ||
Any additional keyword arguments are passed through to the jinja2 | ||
``self.template.render`` process. This is useful when you need to provide | ||
additional variables for a custom template. | ||
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
IIRC aren't we supposed to have a
_repr_latex_
?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't know. See if you prefer the new pattern, simple change.