Skip to content

fix: resolve merge conflicts from develop consolidation#65

Open
Goldokpa wants to merge 1 commit into
mainfrom
fix/resolve-merge-conflicts
Open

fix: resolve merge conflicts from develop consolidation#65
Goldokpa wants to merge 1 commit into
mainfrom
fix/resolve-merge-conflicts

Conversation

@Goldokpa
Copy link
Copy Markdown
Member

Summary

  • Resolves all 6 merge conflict regions across 3 files left over from the develop→main consolidation (PR chore: consolidate develop into main (single-trunk) #63)
  • auth.py: Kept cv_dev dev bypass, removed duplicate comment
  • main.py: Merged governance/flooding/SHAP imports, fixed duplicate org parameter crash in predict_upload, added missing numpy import, kept dict[str, Any] type annotations
  • pipeline.py: Adopted torch.nn.Module return type for ONNX flexibility, removed synthetic fallback logic in favor of clean GEE production path

Test plan

  • CI passes (Python compile + existing test suite)
  • predict_upload endpoint no longer crashes on import (duplicate param fix)
  • Flooding prediction path works with FloodingAnalysis import resolved
  • ONNX inference path compatible with torch.nn.Module return type

🤖 Generated with Claude Code

Resolves 6 conflict regions across auth.py, main.py, and pipeline.py
from the develop→main single-trunk consolidation. Keeps dev bypass in
auth, merges governance/flooding imports, fixes duplicate org param in
predict_upload, adds missing numpy import, and adopts clean GEE
production path (no synthetic fallbacks).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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.

1 participant