McMaster-Baja-Racing / McMaster-Baja-Racing/Better-Data-Viewer
Autozoom Video Sync
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 3
- Forks
- 2
- PR merge metrics
- No merged PRs in 30d
Description
Currently when a new video sync chart is created, you have to manually zoom into the chart to line it up with the video. This causes a lot of slowdown since a lot of the chart has to be loaded which isn't used. This should be changed so that the chart is zoomed automatically so that only the portion of the chart that overlaps with the video is shown. Until zoom is fixed on non-timestamp graphs, this should only happen on charts that have timestamp on the x-axis
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start at the code that creates a new video sync chart and handles timestamp x-axis ranges. Trace how the video interval is available to chart zooming, then verify that only the overlapping portion is shown while non-timestamp charts remain unchanged. No file or test is named, so identify the relevant entry point and existing chart checks first.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- data-visualization, frontend
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100