-
Notifications
You must be signed in to change notification settings - Fork 1
Update Processing Lambda to Support PADRE Craft #1
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
Update Processing Lambda to Support PADRE Craft #1
Conversation
There was a problem hiding this 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 adds support for the PADRE Craft instrument to the file processing Lambda function. The changes enable processing of PADRE Craft CSV data files alongside existing instrument support, with improved handling of calibration outputs that may include None values.
Key Changes:
- Added PADRE Craft instrument package dependency and test data
- Refactored file calibration logic to return lists of files and handle None values
- Updated file metadata tracker with improved error handling, type hints, and documentation
- Modified CI/CD workflow and README to demonstrate PADRE-specific testing
Reviewed Changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| lambda_function/tests/test_data/test_craft_event.json | New test event for PADRE Craft S3 bucket triggering Lambda processing |
| lambda_function/tests/test_data/padre_get_CUBEADCS_GEN2_OP_STATUS_APP_Data_1761936771334_1762106179414.csv | Sample PADRE Craft CSV telemetry data for testing |
| lambda_function/src/file_processor/file_processor.py | Major refactoring: added list-based file returns, None value handling, CSV support, keyword arguments for metatracker, and improved documentation |
| lambda_function/padre-requirements.txt | Added padre_craft package dependency |
| README.md | Updated docker command examples with multi-line formatting and corrected repository link |
| .github/workflows/calibration.yml | Updated workflow to test PADRE mission with new base image and test data |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
No description provided.