-
Notifications
You must be signed in to change notification settings - Fork 7
Worker implementation for processing images in the Antenna task queue (PSv2) #94
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from 32 commits
Commits
Show all changes
46 commits
Select commit
Hold shift + click to select a range
9129421
WIP: Pull worker and REST dataset
carlosgjs 41fef93
Clean-up, addd "worker" cli command, move token to env var
carlosgjs 87910aa
Post results back
carlosgjs c67afce
Progress updates working
carlosgjs 64e188d
clean up
carlosgjs c00de9d
Better error handling
carlosgjs 3b60538
Support multiple pipelines
carlosgjs 45e68bc
Use app.state for the service info
carlosgjs 3c4dd8c
API launch target
carlosgjs 8f76365
Integration fixes
carlosgjs bef1cd7
Use PipelineProcessingTask instead of raw dicts
carlosgjs 52cff32
Fix to returned results
carlos-irreverentlabs f3f3cd6
Trigger CI workflows
mihow 589cd0d
Add Antenna API settings for worker configuration
mihow c4147bd
Add Pydantic schemas for Antenna API responses
mihow f7f454a
Refactor worker to use Settings pattern and improve robustness
mihow 7846510
Improve datasets error handling and API contract
mihow 822c436
Add type annotations to update_detection_classification
mihow 2f26e0f
Add Antenna worker documentation
mihow 99e685e
Update poetry.lock with dependency updates
mihow ab073b3
Replace fragile urljoin with explicit f-string URL construction
mihow 078aa26
Use plural names for batch dict keys containing lists
mihow 38942ee
Merge branch 'main' of https://github.com/RolnickLab/ami-data-manager…
mihow ce1d754
Fix API tests not running in main test suite
mihow 29172d7
Rename batch result schemas to use Antenna prefix for consistency
mihow d85bafb
turn off typer show locals
carlosgjs 22c4182
add back help text
carlosgjs a30ffd5
Flake fixes
carlosgjs 5baab55
Fix REST dataloader to use localization_batch_size for inference batc…
mihow 1bf5ee5
Fix type annotations to use explicit | None syntax
mihow 1a523b2
Retry worker API requests with urllib3 adapter, reuse sessions (#104)
mihow 9bd7142
AMI: Pipeline Registration (#106)
mihow 602b2bc
Address code review feedback
mihow b1b184c
Disable POST retries by default in get_http_session
mihow ce3d967
Add validation and error handling improvements
mihow 15d07c4
Remove redundant worker tests
mihow 15da4dd
Refactor: Extract Antenna integration into dedicated module
mihow 3825517
chore: remove temporary plans
mihow 8e9c7fb
Simplify HTTP session config: hardcode retry, pass auth explicitly
mihow 1c5ed89
feat: add example service file for Antenna worker, add comments
mihow b427ed2
fix: guard torch.cuda.empty_cache() calls with is_available() check
mihow 2cc0259
fix: use sys.executable for pytest subprocess call
mihow 2594bf3
fix: handle post_batch_results failure to prevent silent data loss
mihow 4598278
refactor: make 'ami worker' the default command, use singular --pipel…
mihow 361da2a
fix: handle post_batch_results failure to prevent silent data loss
mihow c4df11c
chore: remove validate_dwc_export.py (not meant for this PR)
mihow File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -142,3 +142,6 @@ db_data/ | |
| # Test files | ||
| sample_images | ||
| bak | ||
|
|
||
| # Local scratch for moving untracked files | ||
| scratch/ | ||
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,29 @@ | ||
| { | ||
| // Use IntelliSense to learn about possible attributes. | ||
| // Hover to view descriptions of existing attributes. | ||
| // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387 | ||
| "version": "0.2.0", | ||
| "configurations": [ | ||
| { | ||
| "name": "Python Debugger: Current File", | ||
| "type": "debugpy", | ||
| "request": "launch", | ||
| "program": "${file}", | ||
| "console": "integratedTerminal" | ||
| }, | ||
| { | ||
| "name": "Run worker", | ||
| "type": "debugpy", | ||
| "request": "launch", | ||
| "module": "trapdata.cli.base", | ||
| "args": ["worker"] | ||
| }, | ||
| { | ||
| "name": "Run api", | ||
| "type": "debugpy", | ||
| "request": "launch", | ||
| "module": "trapdata.cli.base", | ||
| "args": ["api"] | ||
| } | ||
| ] | ||
| } |
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
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
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
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
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.