Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Passthrough overlay image for waveform summary and overview #7264

Closed
mixxxbot opened this issue Aug 22, 2022 · 12 comments
Closed

Passthrough overlay image for waveform summary and overview #7264

mixxxbot opened this issue Aug 22, 2022 · 12 comments

Comments

@mixxxbot
Copy link
Collaborator

Reported by: rryan
Date: 2014-01-14T20:44:05Z
Status: Fix Released
Importance: Low
Launchpad Issue: lp1269148
Tags: passthrough, polish


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?).

@mixxxbot
Copy link
Collaborator Author

Commented by: esbrandt
Date: 2014-01-15T12:31:33Z


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.

@mixxxbot
Copy link
Collaborator Author

Commented by: daschuer
Date: 2014-01-15T12:49:07Z


Nice Idea! +1

@mixxxbot
Copy link
Collaborator Author

Commented by: rryan
Date: 2014-01-15T15:16:48Z


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.

@mixxxbot
Copy link
Collaborator Author

Commented by: esbrandt
Date: 2014-01-15T17:13:06Z


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

@mixxxbot
Copy link
Collaborator Author

Commented by: rryan
Date: 2014-01-15T18:16:11Z


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).

@mixxxbot
Copy link
Collaborator Author

Commented by: daschuer
Date: 2014-01-15T18:57:56Z


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"

@mixxxbot
Copy link
Collaborator Author

Commented by: Holzhaus
Date: 2019-09-05T11:24:05Z


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.

@mixxxbot
Copy link
Collaborator Author

Commented by: ronso0
Date: 2020-03-31T23:45:56Z


#2575
Show "passthrough" on Waveform Overviews if enabled

@mixxxbot
Copy link
Collaborator Author

Commented by: ronso0
Date: 2020-04-01T02:01:18Z


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.

@mixxxbot
Copy link
Collaborator Author

Commented by: ronso0
Date: 2020-06-13T13:45:05Z


fixed.

Live waveform covered by lp:1882812

@mixxxbot
Copy link
Collaborator Author

Commented by: ronso0
Date: 2022-06-12T11:58:16Z


overlay for waveform:
#4794

@mixxxbot
Copy link
Collaborator Author

Issue closed with status Fix Released.

@mixxxbot mixxxbot transferred this issue from another repository Aug 24, 2022
@mixxxbot mixxxbot added this to the 2.3.0 milestone Aug 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant