David Bieber
David Bieber
Ah, So if I'm understanding things correctly, here's what's happening. DataFrames don't redefine \_\_str\_\_, they just redefine \_\_repr\_\_. The default implementation of \_\_str\_\_ is simply to call \_\_repr\_\_. The fix...
Well, my approach to detecting custom `__str__` methods doesn't seem to work quite as simply for detecting custom `__repr__`. Depending on the type of the object, the default `__repr__` implementation...
That's great -- thanks for sharing that @melsabagh-kw!
As chey noted, this already exists. If there's something about this that could be improved, feel free to reopen the issue or open a new issue with more details about...
Looks good! We'll make some small changes as part of merging, but no need to take additional action on your part. 👍 One point for discussion: You truncate types and...
We ran into an error when merging initially. The long_type test fails internally: `AssertionError: 'POSITIONAL ARGUMENTS\n LONG_OBJ\n Type: typing.Tuple[typing.Tuple[typing.Tuple[typing.Tuple[typing.Tupl...' not found in 'NAME\n long_type\n\nSYNOPSIS\n long_type LONG_OBJ\n\nPOSITIONAL ARGUMENTS\n LONG_OBJ\n Type: Tuple\n\nNOTES\n...
It's Python 3.6.7 that it's failing on, but I think we may be using a nonstandard `typing` library. It passes on Travis, but fails on the Google testing infrastructure. When...
Merged with 79d85df2706b2dab5dd5075f2a76953743ee9bf3
> I finally resolved this issue by replacing the : in the bash script with :. I don't follow -- you replaced a colon with a different colon? It would...
If you sign the CLA, happy to merge this PR.