Skip to content

Drop Train All workflow from WASM app surface#15

Draft
Copilot wants to merge 2 commits into
mainfrom
copilot/drop-train-all-pruning
Draft

Drop Train All workflow from WASM app surface#15
Copilot wants to merge 2 commits into
mainfrom
copilot/drop-train-all-pruning

Conversation

Copy link
Copy Markdown

Copilot AI commented May 26, 2026

“Train All” is disproportionately expensive in the browser/WASM runtime and does not fit the app’s execution model. This PR removes that path and keeps training scoped to explicitly selected models.

  • UI: remove bulk-train affordance

    • Removed the Train All toolbar button from frontend/index.html.
  • Frontend behavior: remove train-all code path

    • Deleted trainAllModels() from frontend/js/app.js.
    • Removed trainAllBtn state toggling from updateTrainButtons() so only Train Selected is controlled.
  • Python bridge/API surface: prune unused endpoint

    • Removed train_all(...) from frontend/py/learner.py to match the new UI capabilities and reduce unnecessary backend surface.
  • Internal docs alignment

    • Updated CLAUDE.md function list to no longer reference train_all.
// before: toggled both train buttons
document.getElementById('trainBtn').disabled = !(hasData && hasTarget && hasFeatures && hasSelectedModels);
document.getElementById('trainAllBtn').disabled = !(hasData && hasTarget && hasFeatures);

// after: only selected-model training remains
document.getElementById('trainBtn').disabled = !(hasData && hasTarget && hasFeatures && hasSelectedModels);

Copilot AI linked an issue May 26, 2026 that may be closed by this pull request
Copilot AI changed the title [WIP] Prune 'train all' functionality for web assembly Drop Train All workflow from WASM app surface May 26, 2026
Copilot AI requested a review from yanndebray May 26, 2026 17:36
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.

Drop Train All

2 participants