Passthrough overlay image for waveform summary and overview

Bug #1269148 reported by RJ Skerry-Ryan
26
This bug affects 4 people
Affects Status Importance Assigned to Milestone
Mixxx
Fix Released
Low
Jan Holthuis

Bug Description

Maybe once Bug #1180872 is solved the overview and summary will support a "disabled" pixmap so we could draw something on the waveform that says "PASSTHROUGH ENABLED" or something like that.

We could also do this with skin XML to hide the waveform, spinny, and overview and replace it with label widgets or something else (a BIG vu meter?).

RJ Skerry-Ryan (rryan)
Changed in mixxx:
milestone: none → 1.12.0
assignee: nobody → jus (jus)
Revision history for this message
jus (jus) wrote :

I`m not sure anymore if disabling summary and overview is really the best way to do it. You would have to eject a playing track eventually and whatnot.

E.g. in another soft they just display an text overlay on the affected deck and mute the internal playback. This is pretty nice if you running short on decks (think 2-deck setup) because you can always see whats going on the deck, switch back to internal playback at exact position, and even load tracks to a deck with passthrough enabled.

tags: added: passthrough
Revision history for this message
Daniel Schürmann (daschuer) wrote :

Nice Idea! +1

Revision history for this message
RJ Skerry-Ryan (rryan) wrote :

Ah, I like that too. Currently the EngineBuffer doesn't get processed if passthrough is enabled but if we did that then the deck would still work. So we should add a passthrough overlay pixmap to the summary and overview that is drawn on top when passthrough is enabled.

summary: - Waveform summary and overview should disable when passthrough enabled
+ Passthrough overlay image for waveform summary and overview
Changed in mixxx:
assignee: jus (jus) → nobody
Revision history for this message
jus (jus) wrote :

Drawing the overlays blocks clicking on the waveform widgets, right?
Any way around this?

Revision history for this message
RJ Skerry-Ryan (rryan) wrote :

The implementation I'm imagining would not block mouse input to the widget -- it would just draw the overlay on top (not like the BPM tap button blocking mouse input to the WLabel below it).

Revision history for this message
Daniel Schürmann (daschuer) wrote :

We currently have non blocking "Overlay" text: "Loading track .."
https://github.com/mixxxdj/mixxx/blob/master/src/widget/woverview.cpp#L330

We can do is similar for "Passthough enabled"

RJ Skerry-Ryan (rryan)
Changed in mixxx:
importance: Undecided → Low
tags: added: polish
Owen Williams (ywwg)
Changed in mixxx:
milestone: 1.12.0 → 2.1
Be (be.ing)
Changed in mixxx:
status: New → Fix Committed
status: Fix Committed → Confirmed
Be (be.ing)
Changed in mixxx:
milestone: 2.1.0 → none
Revision history for this message
Jan Holthuis (holthuis-jan) wrote :

Actually, it would be even cooler to have a live waveform overview for the audio that is passed through (like the waveform you get when recording audio in Audacity: https://youtu.be/vs92BVVL6V4?t=59).
In that case, the waveform of the loaded track would not be displayed at all, so that #1567645 would be fixed by this as well.

Revision history for this message
ronso0 (ronso0) wrote :

https://github.com/mixxxdj/mixxx/pull/2575
Show "passthrough" on Waveform Overviews if enabled

Revision history for this message
ronso0 (ronso0) wrote :

building on top of #2575 I'll change the 'Passthrough' overlay to a QLabel that can be styled independently from other overview texts (Loading, Analyzing ...).
Hope it's easy to make that translucent, stack it on top of the overview and allow mouse events on the overview. This would suffice the 'short on decks' usecase, wouldn't it?

When someone manages to implement a live overview other features like live recording and then manipulating the recording are obvious, like lp:1407222, and we can add a Deck / Live switch that swaps Deck / Live overview & controls.

Revision history for this message
ronso0 (ronso0) wrote :

fixed.

Live waveform covered by lp:1882812

Changed in mixxx:
milestone: none → 2.3.0
status: Confirmed → Fix Committed
assignee: nobody → Jan Holthuis (holthuis-jan)
Changed in mixxx:
status: Fix Committed → Fix Released
Revision history for this message
ronso0 (ronso0) wrote :
Revision history for this message
Swiftb0y (swiftb0y) wrote :

Mixxx now uses GitHub for bug tracking. This bug has been migrated to:
https://github.com/mixxxdj/mixxx/issues/7264

lock status: Metadata changes locked and limited to project staff
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.