-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
DOC: Example missing in pandas.DataFrame.to_html #44945 #54153
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
Conversation
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.
it looks like this example does not pass the test. Could you run it locally to compare outputs? Potentially with assert?
pandas/core/frame.py
Outdated
</tr> | ||
</tbody> | ||
</table> | ||
|
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.
Can remove that line.
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.
Sure can, and thanks for your patience as I sort through these processes. Meanwhile, I have two real dumb questions that I hope you can help with ...
- I looked but I couldnt see where I can see the diff you pasted abov. Where would I have found that?
- I did in fact run the test locally and it appeared to pass. What's the magic incantation so I can run these tests locally before I commit?
OK I lied, I have a third question:
Q3: once I fix appropriately, do I just push into the same branch? I couldnt find a way to generate a new pull request so I assume it automatically knows to rerun the tests on my push, and does it just update the current open pull request?
Thanks again for your patience.
-VV
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.
…ed failing DataFrame.to_html() example
thx @vvaidy |
doc/source/whatsnew/vX.X.X.rst
file if fixing a bug or adding a new feature.Uses the suggestion in the issue report as an example.
@noatamir @phofl SciPy 2023 sprint