Skip to content

feat(argentina): structured errors#75

Draft
outerlook wants to merge 2 commits intomainfrom
feat/arg-error
Draft

feat(argentina): structured errors#75
outerlook wants to merge 2 commits intomainfrom
feat/arg-error

Conversation

@outerlook
Copy link
Contributor

@outerlook outerlook commented Jan 31, 2025

@outerlook outerlook self-assigned this Jan 31, 2025
Copy link
Contributor Author

@outerlook outerlook left a comment

Choose a reason for hiding this comment

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

putting on hold for #74

@outerlook
Copy link
Contributor Author

continuing here

@holdex
Copy link

holdex bot commented Feb 19, 2025

@outerlook
⚠️⚠️⚠️
You must submit the time spent on this PR.
⚠️⚠️⚠️

…PA data processing

- Introduced a structured error handling system with detailed error classes for various processing stages
- Added `ErrorAccumulator` to collect and manage errors during data processing
- Implemented context-based error collection with support for markdown artifact generation
- Enhanced error handling in preprocessing, category mapping, and data validation flows
- Added comprehensive error types covering input validation, data processing, and category mapping scenarios
- Improved error traceability with detailed context and responsibility attribution
- Updated existing modules to leverage the new error handling system
…gentina flows

- Introduce comprehensive error context management with `context.py` and `context_helper.py`
- Implement flexible error context tracking using `ContextProperty` descriptor
- Update error classes to leverage global error context for more dynamic error reporting
- Refactor error accumulation to support more granular and context-aware error handling
- Improve error traceability by adding context attributes like `store_id`, `date`, and `file_key`
- Simplify error creation and reduce redundant context parameters
- Add support for concurrent error context isolation
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.

Problem(argentina): Known errors aren't defined in code

1 participant