Skip to content

fix: auto-scale Y-axis, increase chart window to 200 points, add test…#81

Open
a-lie101 wants to merge 1 commit intoUBCMint:mainfrom
a-lie101:main
Open

fix: auto-scale Y-axis, increase chart window to 200 points, add test…#81
a-lie101 wants to merge 1 commit intoUBCMint:mainfrom
a-lie101:main

Conversation

@a-lie101
Copy link
Contributor

What?

Fixed chart Y-axis scaling and increased data window size for the signal graph chart view.

How?

  • Changed Y-axis domain from hardcoded [0, 100] to ['auto', 'auto'] to reflect real EEG data (±1000)
  • Removed hardcoded ticks prop that was conflicting with auto-scaling
  • Increased useNodeData chart window from 20 to 200 points
  • Added /test-chart page for local CSV testing without a backend connection

Testing

  • Loaded real EEG CSV (export.csv) via test page and confirmed all 4 signals render correctly at proper scale
  • Verified table displays most recent rows correctly

@a-lie101 a-lie101 requested a review from MaddieWright as a code owner March 22, 2026 10:38
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