Skip to content

Keep every table in the text, parsed or not, placed or not - #65

Merged
jdkent merged 2 commits into
neuroquery:mainfrom
jdkent:enh/keep_references_option
Oct 2, 2026
Merged

jdkent merged 2 commits into
neuroquery:mainfrom
jdkent:enh/keep_references_option

Conversation

@jdkent

@jdkent jdkent commented Oct 2, 2026

Copy link
Copy Markdown
Collaborator

Two kinds of table were still lost with keep_tables:

  • A table pubget could not parse had its placeholder removed. It is now rendered from article.xml: its label, tab-separated rows with spanned cells repeated, and its footer.
  • A table in <floats-group> has no placeholder in the body. It is now appended with its caption.

Measured on 300 ns-pond articles: the share whose text holds a coordinate row rose from 64% to 92%. Every remaining miss is a table that is not in article.xml.

🤖 Generated with Claude Code

Two kinds of table were still lost with keep_tables. One pubget could
not parse had its placeholder removed; it is now rendered from
article.xml (label, tab-separated rows with spans repeated, footer). One
in <floats-group> has no placeholder in the body; it is now appended
with its caption. Over 300 ns-pond articles the share whose text holds a
coordinate row rose from 64% to 92%; every remaining miss is a table
that is not in article.xml.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>

@jdkent jdkent left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

LGTM

@jdkent
jdkent merged commit 30cbb9d into neuroquery:main Oct 2, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant