Skip to content

✨ add support for crop & split extractions - #395

Merged
sebO-S merged 4 commits into
mainfrom
add-crop-split-v2
Mar 30, 2026
Merged

sebO-S merged 4 commits into
mainfrom
add-crop-split-v2

Conversation

@sebO-S

@sebO-S sebO-S commented Mar 30, 2026

Copy link
Copy Markdown
Collaborator

Description

  • ✨ add support for crop extraction
  • ✨ add support for split extraction
  • 🐛 fix file saves from crop/split extractors breaking in some instances

Types of changes

  • Docs change / refactoring / dependency upgrade
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Requires a change to the official Guide documentation.

@sebO-S
sebO-S requested a review from ianardee March 30, 2026 12:11

@ianardee ianardee left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Missing the custom class from the ticket:

// class SplitFiles extends ExtractedImage[]
splitFiles.saveAllToDisk("/path/to/output/folder", "filename-prefix");

Comment thread mindee/extraction/common/image_extractor.py Outdated
Comment thread mindee/v2/file_operations/crop.py Outdated
Comment thread mindee/v2/file_operations/split.py Outdated
Comment thread mindee/v2/product/crop/crop_box.py Outdated
Comment thread mindee/v2/product/crop/crop_response.py Outdated
Comment thread mindee/v2/product/split/split_range.py Outdated
Comment thread mindee/v2/product/split/split_response.py Outdated
@sebO-S
sebO-S requested a review from ianardee March 30, 2026 14:11
@sebO-S
sebO-S force-pushed the add-crop-split-v2 branch from 94063fe to 9ba47a9 Compare March 30, 2026 14:40
@sebO-S
sebO-S merged commit 9627f48 into main Mar 30, 2026
28 checks passed
@sebO-S
sebO-S deleted the add-crop-split-v2 branch March 30, 2026 15:45
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