Skip to content

feature(app-overhaul) stabilize routing and add device-wide Headphone EQ + UX refresh#136

Closed
Vudgekek wants to merge 13 commits into
ronitsingh10:mainfrom
Vudgekek:beta
Closed

feature(app-overhaul) stabilize routing and add device-wide Headphone EQ + UX refresh#136
Vudgekek wants to merge 13 commits into
ronitsingh10:mainfrom
Vudgekek:beta

Conversation

@Vudgekek

@Vudgekek Vudgekek commented Mar 6, 2026

Copy link
Copy Markdown

Summary

A broad overhaul of FineTune’s audio pipeline and popup UX.

It improves routing/process stability, removes several long-standing pain points, and adds a more powerful device-focused EQ workflow for everyday use.

Fixes

Features

Testing Passed

  • Tested with 2+ output devices.
  • Tested device hot-plug, including disconnect during active playback.
  • Tested with 5+ apps playing audio simultaneously.

@Vudgekek

Vudgekek commented Mar 8, 2026

Copy link
Copy Markdown
Author

Apologies for previous commit spam, force-pushed here to make it more clear what's going on. Also, initially remade this PR because Github had restricted my account's public visibility (old one had to be closed for some reason).

@ronitsingh10

Copy link
Copy Markdown
Owner

Hey @Vudgekek — thanks for putting in serious work here. There's a lot of valuable stuff in this PR.

The main challenge is that main has been reorganized since this branch was created — files have moved into subdirectories (Audio/Engine/, Audio/EQ/, Audio/Monitors/, etc.), and there's new infrastructure like BiquadProcessor, CrashGuard, SoftLimiter, OrphanedTapCleanup, and TapResources that this PR would overwrite.

The scope also makes it hard to review and merge safely as one unit. Would you be open to rebasing against latest main and splitting into smaller PRs? That way each piece can be reviewed and merged independently.

@Vudgekek

Vudgekek commented Mar 8, 2026

Copy link
Copy Markdown
Author

Sure thing!

@Vudgekek Vudgekek closed this Mar 8, 2026
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.

2 participants