pretty.install() fix for some objects with _repr_*_

[darrenburns]

Type of changes

• Bug fix - Fixes [BUG] repr fails for pd.Series in Jupyter with pretty.install() #1717
• New feature
• Documentation / docstrings
• Tests
• Other

Checklist

• I've run the latest black with default args on new code.
• I've updated CHANGELOG.md and CONTRIBUTORS.md where appropriate.
• I've added tests for new code.
• I accept that @willmcgugan may be pedantic in the code review.

Description

The issue is that Rich assumes that if a _repr_*_ method exists on an object, then notebook will definitely render that object.

However, notebook only supports a subset of mimetypes, and even in the case of supported mimetypes, some libraries (e.g. pandas) support toggling off functionality, which results in the _repr_*_ for that mimetype returning None.

For example, pd.Series has a _repr_latex_ method, but by default the config for it ("display.latex.repr") is switched off and so pd.Series._repr_latex_ returns None.

We need something a little finer-grained and more aware of the notebook capabilities.

For all of the mimetypes that Jupyter notebook can display, we check if the corresponding _repr_*_ special method exists and ensure it doesn't return None. If it returns None, Rich will print it, otherwise the notebook will render it.

[willmcgugan]

LGTM - Could you add that to CHANGELOG.md

[willmcgugan]

👍

[xoolive]

So cool, many thanks to you both.
However, on my side, the pd.Series now appears (:heart_on_fire:) but is not pretty printed. That's really not a big deal (better than printing nothing) but just saying... :man_shrugging:

[willmcgugan]

That's likely because pd.Series has support for advanced rendering in Jupyter. We don't want to use Rich's printing if the object already knows how to format itself.

[xoolive]

Fine with me, thanks anyway!