Add support for the Epson V700 Scanner - #56
Merged
Conversation
Register the new instrument enum, wire it into the Lambda handler, and add a TIFF-to-JPEG processing pipeline that resizes high-resolution scans to web-friendly previews and extracts TIFF metadata. Co-authored-by: Cursor <cursoragent@cursor.com>
S3 event notifications only support a single suffix per rule, so add a second notification block to cover the .tiff extension alongside .tif. Co-authored-by: Cursor <cursoragent@cursor.com>
Contributor
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
lanery
changed the title
Co-authored-by: Cursor <cursoragent@cursor.com>
lanery
commented
May 7, 2026
lanery
left a comment
lanery
left a comment
Contributor
Author
There was a problem hiding this comment.
Added! The epson-scanner command is now available in the data-hub-process CLI:
uv run data-hub-process epson-scanner path/to/scan.tif
uv run data-hub-process epson-scanner path/to/scan.tif --output-dir out/It loads the TIFF via TIFFToJPEGConverter, exports a resized JPEG preview, and prints the extracted TIFF metadata. Also updated the docs (supported instruments table + CLI commands table).
wasimxyz
approved these changes
May 7, 2026
4 tasks
wasimxyz
added a commit
that referenced
this pull request
May 7, 2026
…ersions (#59) * CI: Allow deploy role to update S3 event notifications, bump action versions Adds s3:GetBucketNotification/PutBucketNotification to the SAM deploy role scoped to the three stack-owned data buckets so new instrument triggers can roll out through CI instead of requiring an admin sam deploy. The previous deploy of #56 (Epson V700 Scanner) failed because adding the new LambdaConfigurations entries to RawDataBucket required PutBucketNotification, which the role did not grant. Also bumps every workflow's action plugins to their latest stable major versions (actions/checkout v4 -> v6, setup-node v5 -> v6, setup-uv v6 -> v8, upload-artifact v4 -> v7, download-artifact v4 -> v8, configure-aws-credentials v4 -> v6, setup-sam v2 -> v3) to clear the Node 20 deprecation warnings ahead of GitHub forcing Node 24 in June. Co-authored-by: Cursor <cursoragent@cursor.com> * CI: Pin setup-uv to v8.1.0 (no moving v8 major tag exists yet) The astral-sh/setup-uv repo has published v8.0.0 and v8.1.0 as exact tags but has not yet pushed a moving v8 major tag, so `@v8` failed to resolve on the bumped workflows. Pin to v8.1.0 (the latest stable) until the moving tag is published. Co-authored-by: Cursor <cursoragent@cursor.com> --------- Co-authored-by: Cursor <cursoragent@cursor.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
.tifand.tiffuploads under theepson-v700-scanner/prefix.This is minimal processing for now — the raw TIFF is downsampled to a 1000px JPEG preview and metadata is extracted. Future work will layer on plate detection and then colony detection.
Test plan
.tifscan to verify end-to-end processingMade with Cursor