Skip to content

Fix feedback_store.csv paths, retrain.py syntax errors, and adminSeeder.js typos#230

Merged
Userunknown84 merged 1 commit into
Userunknown84:mainfrom
onkar0127:fix-feedback-paths
Jun 23, 2026
Merged

Fix feedback_store.csv paths, retrain.py syntax errors, and adminSeeder.js typos#230
Userunknown84 merged 1 commit into
Userunknown84:mainfrom
onkar0127:fix-feedback-paths

Conversation

@onkar0127

@onkar0127 onkar0127 commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

fixes #148

Description

This Pull Request standardizes the feedback_store.csv filepath resolution between the Flask API save path, the insights dashboard loader, and the model retraining script. It also cleans up critical merge conflicts/syntax errors in retrain.py and typo bugs in adminSeeder.js.

Changes

  1. Feedback Path Standardization:
    • Updated backend/spam_insights.py to point to the correct output folder: backend/output/feedback_store.csv.
    • Updated backend/retrain.py to resolve default --feedback and --dataset paths relative to the script's directory, enabling execution from any working directory.
  2. retrain.py Syntax & Merge Conflict Fixes:
    • Removed old merge conflict residues (feature/main text fragments).
    • Removed redundant old training code nested inside backup_existing_files() that previously threw an IndentationError.
  3. adminSeeder.js Typo Fixes:
    • Corrected conso;e.log and cnsole.log to console.log to prevent server crashes upon MongoDB connection.

Verification Results

  • Pytest: Executed pytest backend/tests and all 58 tests passed with zero failures.
  • Retrain Script: Ran python backend/retrain.py --help successfully.

@vercel

vercel Bot commented Jun 23, 2026

Copy link
Copy Markdown

@onkar0127 is attempting to deploy a commit to the Aditya Sharma's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions Bot added SSoC26 bug Something isn't working Hard labels Jun 23, 2026
@Userunknown84 Userunknown84 merged commit 3745e3b into Userunknown84:main Jun 23, 2026
5 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working Hard SSoC26

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug: Broken feedback CSV path prevents insights updates and retraining

2 participants