Skip to content

Phase 9: GUI Wiring & Dashboard#110

Closed
4awmy wants to merge 10 commits into
mainfrom
phase-9-gui-wiring
Closed

Phase 9: GUI Wiring & Dashboard#110
4awmy wants to merge 10 commits into
mainfrom
phase-9-gui-wiring

Conversation

@4awmy
Copy link
Copy Markdown
Owner

@4awmy 4awmy commented May 3, 2026

Complete wiring of all solvers, dashboard reorganization, status bar implementation, and consistent styling.

4awmy and others added 10 commits April 30, 2026 11:10
Comprehensive updates across all solvers and GUI pages:

**Engine Improvements:**
- Enhanced root finder with improved iteration tracking
- Extended calculus engine with all numerical integration methods
- Improved network solver with better convergence handling
- Better step-by-step data collection for all solvers

**GUI Enhancements:**
- Ch 2 (Linear Systems): Dynamic n×n matrix grid input, load example problems, iteration table display
- Ch 1 (Root Finding): Load example problems dropdown, iteration table in right panel, fixed input field layout
- Ch 3 (Numerical Calculus): Enhanced layout and visualization
- All pages: Iteration tables showing step-by-step solver progress

**User Experience:**
- Load example buttons allow quick testing with predefined problems from numerical_methods_problems.md
- Iteration tables display convergence progress (iteration #, values, errors)
- Better input validation and error handling
- Improved result display panels

This enables users to:
✅ Solve all problems from numerical methods problem sheet
✅ Use matrix grid for easy linear system entry
✅ View detailed iteration history
✅ Load pre-configured examples for testing

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
@4awmy 4awmy self-assigned this May 3, 2026
@4awmy
Copy link
Copy Markdown
Owner Author

4awmy commented May 3, 2026

Self-review: Dashboard headers match request. Status bar updates correctly. All chapter pages fully wired. @copilot review requested.

@4awmy
Copy link
Copy Markdown
Owner Author

4awmy commented May 3, 2026

Review: Phase 9 — GUI Wiring & Dashboard

@gemini please fix the following issues on branch phase-9-gui-wiring:


VERIFY — All 4 chapter pages use EquationInputWidget, not raw CTkEntry

Check numcore_gui/pages/chapter_1_app.py, chapter_2_app.py, chapter_3_app.py, chapter_4_app.py:

  • Chapter 1 (Root Finding) must use EquationInputWidget for f(x) input
  • Chapter 4 (Integration) must use EquationInputWidget for f(x) input
  • Chapters 2 and 3 use matrix/point inputs (no equation widget needed, but verify no raw CTkEntry is used for expressions)

If any chapter uses a raw CTkEntry for function expressions, replace it with:

from numcore_gui.equation_input import EquationInputWidget
self.eq_input = EquationInputWidget(self, label_text="f(x):")

VERIFY — SmartSolverPanel wired as inline frame, not popup

After PR #109 fix changes SmartSolverPanel to CTkFrame, verify each chapter page instantiates it as:

self.smart_panel = SmartSolverPanel(self)
self.smart_panel.grid(row=X, column=0, sticky="ew", padx=10, pady=10)
self.smart_panel.grid_remove()  # hidden by default

And the Smart Solve button shows it:

def _on_smart_solve(self):
    result = chapter_N_compare(...)
    self.smart_panel.populate(result)
    self.smart_panel.grid()  # show it

BUG — ResultPanel not receiving steps from solver

Verify that after each solver.solve() call, the page also calls:

result_panel.update_result(data, solver.get_steps())

If solver.get_steps() is not called, the methodology table will always be empty.


MISSING — Mission-themed labels not removed from dashboard

File: numcore_gui/dashboard.py

Search and remove all strings containing: "Mission", "Analyze Circuit", "Launch", "Command". Replace with clean academic labels: "Root Finding", "Linear Systems", "Interpolation", "Integration".


CLEANUP — Remove artifact files

Remove gemini_*.log, issue*.md, ISSUES.md from the commit.

@4awmy
Copy link
Copy Markdown
Owner Author

4awmy commented May 3, 2026

Fixed: All mission-themed labels removed. EquationInputWidget fully wired into all pages. @claude please review.

@4awmy
Copy link
Copy Markdown
Owner Author

4awmy commented May 3, 2026

Closing — all changes from this phase are included in the squash-merge of PR #111 (phase-11-final-polish) into main. All review fixes have been applied and verified.

@4awmy 4awmy closed this May 3, 2026
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