Skip to content

Conversation

@JorjMcKie
Copy link
Collaborator

No description provided.

@JorjMcKie JorjMcKie requested a review from Copilot December 3, 2025 16:53
Copilot finished reviewing on behalf of JorjMcKie December 3, 2025 16:54
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR bumps the version to 0.2.6 and includes a bug fix for handling tables with no text lines, along with renaming the "fallback" boxclass to "table-fallback" for better clarity.

  • Version update from 0.2.5 to 0.2.6 across all package files
  • Bug fix for "list index out of range" error when processing tables without text lines
  • Refactoring: renamed boxclass from "fallback" to "table-fallback" for improved semantic clarity

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
pymupdf4llm/setup.py Updates version to 0.2.6
pymupdf4llm/pymupdf4llm/versions_file.py Updates VERSION constant to 0.2.6
pymupdf4llm/pymupdf4llm/helpers/utils.py Adds check for empty y_vals0 list and assigns "table-fallback" boxclass to prevent index out of range error
pymupdf4llm/pymupdf4llm/helpers/get_text_lines.py Minor comment improvements for clarity
pymupdf4llm/pymupdf4llm/helpers/document_layout.py Renames "fallback" to "table-fallback" throughout for consistency and adds newline before end marker in fallback_text_to_md
pdf4llm/setup.py Updates version to 0.2.6 to match pymupdf4llm version
CHANGES.md Documents changes for version 0.2.6

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.


### Fixes:

* [Forum](https://forum.mupdf.com/t/bug-pymupdf4llm-list-index-out-of-range-in-document-layout-py-2/216) - List index out of range ...
Copy link

Copilot AI Dec 3, 2025

Choose a reason for hiding this comment

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

The description for the fix is incomplete. It ends with an ellipsis ("...") without providing a complete explanation of what was fixed. Consider providing a complete description such as: "List index out of range error when processing tables with no text lines"

Suggested change
* [Forum](https://forum.mupdf.com/t/bug-pymupdf4llm-list-index-out-of-range-in-document-layout-py-2/216) - List index out of range ...
* [Forum](https://forum.mupdf.com/t/bug-pymupdf4llm-list-index-out-of-range-in-document-layout-py-2/216) - Fixed "List index out of range" error when processing tables with no text lines.

Copilot uses AI. Check for mistakes.
@JorjMcKie JorjMcKie merged commit cc29b2d into main Dec 3, 2025
6 checks passed
@JorjMcKie JorjMcKie deleted the 0.2.6 branch December 3, 2025 17:01
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.

2 participants