McMaster-Baja-Racing / McMaster-Baja-Racing/Better-Data-Viewer
Streamline Chart Data Parsing + Video Sync
Open
Nobody has claimed this yet.
code quality
frontend
- Dominant language
- TypeScript
- Stars
- 3
- Forks
- 2
- PR merge metrics
- No merged PRs in 30d
Description
There's a lot of pretty repetitive code inside the chart data and video sync lines that could be cleaned up a lot.
- Combining the handling of colour charts and all others requires a lot of duplicated cases, and if we come along with a separate data type for it, it will only multiply. Would be great if this could be separated / abstracted out.
- Similar to chart video sync lines. Currently only have
verticalorvert + horibars, handled with anif. Preferably we have separate flows for them
Other than that, code is just messy.
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 by locating the chart data parsing and chart video sync line handling, then compare the colour-chart versus other data paths and the vertical versus vert + hori bar paths. Done means the repeated cases are separated or abstracted without changing existing chart and video-sync behavior; no files or tests are named in the issue.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- data-visualization
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100