Skip to content

Persist verify outcomes to SQLite#25

Merged
ionfwsrijan merged 2 commits into
ionfwsrijan:mainfrom
sasidaran-99:persist-verify-outcomes
Jun 3, 2026
Merged

Persist verify outcomes to SQLite#25
ionfwsrijan merged 2 commits into
ionfwsrijan:mainfrom
sasidaran-99:persist-verify-outcomes

Conversation

@sasidaran-99
Copy link
Copy Markdown
Contributor

Before opening: make sure there is an issue tracking this work, and link it below. PRs without a linked issue may be closed without review.

Linked issue

Closes #

What this PR does

Type of change

  • Bug fix
  • New feature
  • ML model / training pipeline
  • Refactor (no behaviour change)
  • Documentation
  • Tests only

ML tier (if applicable)

  • Tier 1 — Triage
  • Tier 2 — Predictive
  • Tier 3 — Autonomous
  • Not ML-related

Changes

Backend

Frontend

New dependencies

Database / schema changes


Testing

How did you test this?

Checklist

  • Tested locally end-to-end (upload ZIP or GitHub URL → scan → findings returned correctly)
  • New ML model falls back gracefully when model file is absent
  • No new console.error or unhandled Python exceptions introduced
  • Added or updated tests where applicable
  • requirements.txt / package.json updated if new dependencies added
  • New model files (.pkl, .pt, etc.) are gitignored, not committed

Anything reviewers should focus on

Screenshots (if UI changed)

@sasidaran-99
Copy link
Copy Markdown
Contributor Author

@ionfwsrijan

I've completed the implementation for this issue, verified the functionality locally, and addressed the CI formatting feedback. All checks are now passing.

Could you please review the PR when you have a chance? If everything looks good, I'd appreciate it being merged.

@rushi-k12
Copy link
Copy Markdown

@ionfwsrijan it seems fine, go ahead

@ionfwsrijan ionfwsrijan merged commit cba7a8c into ionfwsrijan:main Jun 3, 2026
6 checks passed
@ionfwsrijan ionfwsrijan added backend Backend issues easy Easy difficulty SSoC26 labels Jun 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend Backend issues easy Easy difficulty SSoC26

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants