Skip to content

BUG: fix duplicate entries in LaTeX List of Tables when using longtable environments #36297

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 9 commits into from
Sep 19, 2020

Conversation

jeschwar
Copy link
Contributor

@jreback
Copy link
Contributor

jreback commented Sep 11, 2020

this should target 1.2 as there was a large refactor in this code (which u already did), just move the note

Copy link
Contributor

@jreback jreback left a comment

Choose a reason for hiding this comment

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

see
comments

@ivanovmg ivanovmg added this to the 1.2 milestone Sep 14, 2020
@jreback jreback added the IO LaTeX to_latex label Sep 19, 2020
@jreback
Copy link
Contributor

jreback commented Sep 19, 2020

@ivanovmg if you'd have a look.

@ivanovmg
Copy link
Member

@ivanovmg if you'd have a look.

@jreback, this looks good to me. However, if there are other modifications to longtable in future, we may want to redefine the method get_result for LongTableBuilder, where we specify more specific order of the items. Note that this PR suggests to include header in the middle separator. This is not very obvious for the reader. But the author left the comment why it is there, so it is OK for now.

@jreback
Copy link
Contributor

jreback commented Sep 19, 2020

lgtm ping on green.

@jreback jreback merged commit 475a980 into pandas-dev:master Sep 19, 2020
@jreback
Copy link
Contributor

jreback commented Sep 19, 2020

thanks @jeschwar

kesmit13 pushed a commit to kesmit13/pandas that referenced this pull request Nov 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
IO LaTeX to_latex
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ENH: Correct use of longtable, label, and caption with LaTeX
3 participants