ENH: visual overhaul, macaca templates, and new validation features#252
Open
ENH: visual overhaul, macaca templates, and new validation features#252
Conversation
UI/Visual: - Complete CSS redesign with Space Mono font, scanline overlay, glow animations, and consistent dark/green design system - Responsive hero page with live template stats, how-it-works workflow, feature grid, and template browser - Validator page redesigned: drag-and-drop file zone, styled selects, animated submit button, dev-mode login notice - Larger AFIDs banner (clamp 360–620px) Validation features: - Regional radar chart (error profile across 8 neuroanatomical regions) - Directional error language per landmark (e.g. "2.1mm posterior") - Session summary dashboard: mean±SD, best/worst landmark, within-2mm count - Color-coded distance table (green/amber/red by threshold) Macaca support: - Add afids-templates/macaca/ with 6 mean consensus FCSV templates: D99, INIA19, MacaqueMNI, NMTv1.3, NMTv2.0asym, Yerkes19 - Templates sourced from afids/afids-macaca PHASE1_output_afid - Species dropdown now includes Macaca alongside Human
Dhananjhay
reviewed
Mar 18, 2026
| <span class="stat-green">{{ summary.n_within_2mm }}</span> | ||
| <span style="font-size:1rem; color:var(--muted);">/{{ summary.total }}</span> | ||
| </div> | ||
| <div class="summary-stat-sub">{{ summary.n_within_1mm }} within 1mm</div> |
There was a problem hiding this comment.
This line appears after the summary which is a little confusing -- consider moving it after 'WITHIN 2 MM' sentence
Dhananjhay
reviewed
Mar 18, 2026
| <span style="font-size:1rem; color:var(--muted);">/{{ summary.total }}</span> | ||
| </div> | ||
| <div class="summary-stat-sub">{{ summary.n_within_1mm }} within 1mm</div> | ||
| <div class="summary-stat-label">Within 2mm</div> |
There was a problem hiding this comment.
The mm appears as MM in the GUI -- better if it stays lowercase
|
It looks really good! Left some minor comments + revision notes but overall should be good to be merged. I tested the new design using a sample fscv and it worked just fine. Amazing work @ataha24! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
UI/Visual:
Validation features:
Macaca support:
Proposed changes
Describe the changes implemented in this pull request. If the changes fix a bug or resolves a feature request, be sure to link the issue. Please explain the issue and how the changes address the issue.
Types of changes
What types of changes does your code introduce? Put an
xin the boxes that applyChecklist
Put an
xin the boxes that apply. You can also fill these out after creating the PR. If you are unsure about any of the choices, don't hesitate to ask!blackwith the-l 79flag.Notes
All PRs will undergo the unit testing before being reviewed. You may be requested to explain or make additional changes before the PR is accepted.
PR template was adopted from appium