Merge gh pages into main#2
Conversation
There was a problem hiding this comment.
Pull request overview
This PR appears to bring GitHub Pages–oriented presentation changes into main, alongside refreshed Temporal Calc output artifacts and some console/log formatting cleanup across the Temporal and GNN pipelines.
Changes:
- Simplifies/normalizes console banner output in shell + Python scripts for Temporal and GNN pipelines.
- Updates GNN dashboard copy/structure (header text + navigation label changes) and tweaks results summary heading.
- Refreshes committed Temporal Calc output artifacts (CSV/JSON/log) with new run results.
Reviewed changes
Copilot reviewed 12 out of 18 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| temporal_calc/run_all.sh | Removes some decorative separator echo lines from pipeline output. |
| temporal_calc/outputs/neighborhood_risk_forecast.csv | Updates forecast output values. |
| temporal_calc/outputs/model_results.json | Updates model metric summary values. |
| temporal_calc/outputs/logs/max_performance.log | Updates training log content/metrics from a new run. |
| temporal_calc/optimized_training.py | Replaces large banner comments with simpler section headers. |
| temporal_calc/max_performance.py | Replaces large banner comments with simpler section headers. |
| temporal_calc/config.py | Removes banner-style separator comments. |
| temporal_calc/pycache/optimized_training.cpython-313.pyc | Adds a compiled Python cache artifact. |
| GNN/src/main.py | Simplifies step header printing throughout pipeline execution. |
| GNN/src/evaluate.py | Simplifies evaluation header/footer printing. |
| GNN/src/data_split.py | Simplifies split summary header/footer printing. |
| GNN/src/baseline_comparison.py | Simplifies baseline comparison header printing. |
| GNN/dashboard.html | Updates page header text and changes nav button target/label; various formatting edits. |
| GNN/RESULTS_SUMMARY.md | Changes section heading to “Performance Winner…” wording. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| <button class="nav-btn active" onclick="showSection('overview')">Overview</button> | ||
| <button class="nav-btn" onclick="showSection('map')">Interactive Map</button> | ||
| <button class="nav-btn" onclick="showSection('baselines')">Baseline Comparison</button> | ||
| <button class="nav-btn" onclick="showSection('analysis')">Analysis</button> |
There was a problem hiding this comment.
The nav button now calls showSection('analysis'), but the page does not define a <section id="analysis"> (the existing sections include hotspots and baselines). Clicking this button will throw at document.getElementById(sectionId).classList.add('active') and break navigation. Rename the target to an existing section id (e.g., baselines/hotspots) or rename the corresponding section id and update the showSection logic that special-cases baselines.
| <button class="nav-btn" onclick="showSection('analysis')">Analysis</button> | |
| <button class="nav-btn" onclick="showSection('evaluation')">Analysis</button> |
No description provided.