You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I'd be happy to make these changes, but wonder if there are reasons against doing this, before making a PR.
There aren't any reasons against doing this, it just wasn't a priority. You can go ahead and open a PR! Do check out the contributor's guide before opening one.
symroe
added a commit
to symroe/camelot
that referenced
this issue
Feb 25, 2019
I would expect the
TableList
returned byread_pdf
to return a list of tables in the order defined in the PDF (by page, then table on that page).I think this requires a few things:
__lt__
method onTable
that looks like:Create an
__iter__
method onTableList
return sorted(tables)
inread_pdf
I'd be happy to make these changes, but wonder if there are reasons against doing this, before making a PR.
The text was updated successfully, but these errors were encountered: