1bc9e6aaf305b60088bb11d3fb52f5acbf6f9fd4
Bugfix/generic dynamic jan26 * Refactor: Extract dynamic primary metrics logic to separate module - Created src/testbed/dynamic_primary_metrics.py with all dynamic primary field analysis logic - Moved tally-based analysis functions to new module for better modularity - Updated testbed_utils.py to import and use functions from dynamic_primary_metrics - Simplified testbed_metrics_dynamic_only.py to use new module - Removed unused analyze_dynamic_primary_fields function (400+ lines) - Added field filtering in match_rows to handle missing columns gracefully - Minimal changes to testbed_utils.py (~100 lines vs 845 before) * feat: Add debug print blocks and prompt changes for dynamic primary fields in 1:1 processing This commit adds prompt changes for when we pass dynamic primary to 1:1 and comprehensive debug logging for dynamic primary fields when they are escalated to 1:1 processing via Hybrid Smart Chunking (HSC) or Full Context methods. Changes: - Added debug print blocks in hybrid_smart_chunking_funcs.py to log: * Retrieval question, context chunks, and final prompt for HSC processing * Raw LLM output (reasoning + answer) for dynamic primary fields - Added debug print blocks in prompt_calls.py to log: * Contract context, final prompt for Full Context processing * Raw LLM output (reasoning + answer) for dynamic primary fields - Added FULL_CONTEXT_DYNAMIC_PRIMARY_INSTRUCTION() to prompt_templates.py: * Provides specific guidance for dynamic primary fields in full context * Emphasizes extracting values only if they refer to entire contract * Includes pipe-delimited formatting instructions - Updated dynamic_fun… * Ran black, for formatting * Refine dynamic primary field prompts and add Duals auto-detection - Enhanced DYNAMIC_ASSIGNMENT_INSTRUCTION with structural boundary rules - Added exhibit header and subsection context hierarchy for LOB assignment - Implemented Duals auto-detection in dynamic_primary discovery (Medicare+Medicaid -> Duals) - Added Duals value formatting instruction to prevent oversimplification - Commented out DUAL_LOB_CHECK for performance testing - Removed verbose debug print blocks from HSC and Full Context processing - Added focused debug print for dynamic assignment raw LLM output * Remove debug print blocks from pipeline files - Removed all debug print blocks from file_processing.py (8 blocks) - Removed all debug print blocks from dynamic_funcs.py (8 blocks) - Removed debug print block from prompt_calls.py (dynamic assignment) - DUAL_LOB_CHECK remains commented as requested - Total: 188 lines of debug code removed * Merge remote-tracking branch 'origin/main' into bugfix/generic_dynamic_jan26 * Ran black for CI * Remove update_lob_for_duals * Merge branch 'DEV' into bugfix/generic_dynamic_jan26 Approved-by: Katon Minhas
Field Extraction Pipeline
Contract field extraction using LLMs.
Setup
Install uv (if not already installed)
# Ubuntu/WSL (recommended)
curl -LsSf https://astral.sh/uv/install.sh | sh
source ~/.bashrc # or restart terminal
# macOS
curl -LsSf https://astral.sh/uv/install.sh | sh
# Windows (PowerShell - native, not WSL)
powershell -ExecutionPolicy ByPass -c "irm https://astral.sh/uv/install.ps1 | iex"
Verify installation: uv --version
Install dependencies
uv sync
Running the Code
Use uv run with Python's module flag from the project root:
# Run the default SaaS pipeline
uv run python -m src.pipelines.saas.main
# Run with unified runner (supports client selection)
uv run python -m src.pipelines.runner --client saas --input-dir /path/to/input
# Run with specific client
uv run python -m src.pipelines.runner --client clover --input-dir /path/to/input
Note: Always use uv run to ensure correct virtual environment. Always use -m flag for proper imports.
Development
Before pushing, always run these checks to avoid breaking the CI pipeline:
# Format code
uv run black src/
# Type checking
uv run mypy src/
# Run tests
uv run pytest
All checks must pass before merging to main.
Project Structure
├── src/
│ ├── pipelines/ # Pipeline implementations
│ │ ├── runner.py # Unified CLI entry point with client routing
│ │ ├── saas/ # Default SaaS pipeline
│ │ │ └── main.py # Main entry point for SaaS
│ │ ├── shared/ # Shared pipeline components
│ │ │ ├── preprocessing/ # Document preprocessing
│ │ │ ├── extraction/ # Field extraction logic
│ │ │ └── postprocessing/ # Result postprocessing
│ │ └── clients/ # Client-specific overrides
│ │ └── clover/ # Clover client customizations
│ ├── core/ # Core utilities (registry, fieldset)
│ ├── constants/ # Constants, mappings, and field definitions
│ │ ├── mappings/ # Crosswalk JSON files
│ │ └── lists/ # Lookup lists
│ ├── prompts/ # LLM prompt templates
│ ├── utils/ # Shared utilities (IO, string, logging, etc.)
│ ├── codes/ # Medical code extraction utilities
│ ├── crosswalk/ # Crosswalk mapping logic
│ ├── embeddings/ # Pre-computed embeddings for code matching
│ ├── qc_qa/ # QC/QA validation pipeline
│ ├── parent_child/ # Parent-child relationship mapping
│ ├── document_classification/ # Document type classification (DTC)
│ └── tests/ # Unit tests
├── documentation/ # Project documentation
├── outputs/ # Pipeline output files (gitignored)
├── logs/ # Log files (gitignored)
└── pyproject.toml # Project dependencies (uv/pip)
Branching
Naming Conventions
| Type | Pattern | Use Case | Example |
|---|---|---|---|
| Feature | feature/<ticket>-<description> |
New functionality | feature/PROJ-123-add-export-csv |
| Bugfix | bugfix/<ticket>-<description> |
Bug fixes | bugfix/PROJ-456-fix-null-handling |
| Hotfix | hotfix/<ticket>-<description> |
Urgent production fixes | hotfix/PROJ-789-critical-parse-error |
| Test | test/<description> |
Testing/experimentation | test/lesser-table-caching-refactor |
| Release | release/<version> |
Release preparation | release/v1.2.0 |
Branch Guidelines
- Use lowercase with hyphens (kebab-case) for descriptions
- Include ticket number when applicable (e.g., JIRA, GitHub issue)
- Keep branch names concise but descriptive
- Delete branches after merging
Workflow
- Create branch from
main - Make changes and commit with clear messages
- Run checks before pushing:
uv run black src/ && uv run mypy src/ && uv run pytest - Create PR to
main - Ensure all CI checks pass
- Get code review approval
- Squash and merge
Description
Languages
Python
80.5%
Jupyter Notebook
13%
HCL
3.1%
HTML
1.6%
PLpgSQL
1.5%
Other
0.2%