Skip to content

Task 3b: Add solar-panel-defects detection use case - #8

Open
johyeongseob wants to merge 3 commits into
intel:mainfrom
johyeongseob:task3b-solar-panel-defects
Open

Task 3b: Add solar-panel-defects detection use case#8
johyeongseob wants to merge 3 commits into
intel:mainfrom
johyeongseob:task3b-solar-panel-defects

Conversation

@johyeongseob

Copy link
Copy Markdown
Contributor

Summary

This PR adds the Task 3b Dataset A1 solar_panel_defects use case based on the PVEL-AD electroluminescence image dataset.

Changes

  • Add the solar_panel_defects use-case configuration, policy, prompts, and predefined questions.
  • Add OpenVINODetectHandler for single-channel YOLO-style OpenVINO detection output.
  • Decode [1, 16, 8400] output and apply confidence filtering and class-aware NMS.
  • Restore bounding boxes to the original image coordinates.
  • Store detection results in JSONL and SQLite with modality_source: electroluminescence.
  • Integrate the standard policy, analysis, and evidence agents.
  • Add CLI and Web UI chat support.
  • Add handler tests and use-case documentation.

Validation

  • 13 passed across Solar Panel Defects and Oil & Gas regression tests.
  • Verified inference on 100 PVEL-AD test images:
    • 217 raw detections
    • 131 policy-kept detections
    • 86 filtered detections
  • Verified Analysis, Evidence, and SQL chat modes in both CLI and Web UI.
  • Evaluated the full 19,150-image PVEL-AD test set:
    • Precision: 0.8789
    • Recall: 0.7822
    • F1: 0.8277

Notes

The PVEL-AD dataset, OpenVINO model files, and local evaluation script are excluded from this PR.

Thank you,
Hyeongseob Jo

Comment thread src/agents/analysis_agent.py Outdated
) -> str:
"""Generate summary using LLM."""
concise_instruction = """
if use_case == "oil_gas_pipeline":

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Avoid hard-coded components in the agent.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Thanks for the feedback.

I removed the hard-coded use-case logic and moved the report rules to each use-case prompt. All tests passed.

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.

2 participants