Rename master/headphone volume to gain.

Bug #1306253 reported by Daniel Schürmann
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mixxx
Fix Released
Low
RJ Skerry-Ryan

Bug Description

Since [Master],volume is a multiplier to the samples, it behaves like a bit crusher.

The name implies, that is is a volume control of the master output. Instead it is an output gain.

If people using it for lets say play background music during a dinner, it is actually a bitcrusher.

Related post:
http://music.columbia.edu/pipermail/portaudio/2011-January/011366.html

Revision history for this message
Daniel Schürmann (daschuer) wrote : Re: [Master],volume is a bitcursher use hadware mixer instead

Bug #1280771 issues the input mixer

summary: - [Master],volume is a bitcursher
+ [Master],volume is a bitcursher use hadware mixer instead
Revision history for this message
RJ Skerry-Ryan (rryan) wrote :

So should we deprecate the volume control, rename it to "gain" and alias "volume" to it for backwards compatibility?

Your bug title suggests we should remove master gain -- sure people should ideally not reduce the fidelity of their music before it gets converted to analog (unless it's clipping) but the live broadcasting and recording crowd at least need a handy gain knob. And many DJs will want it for the convenience even though it is less ideal than doing it with the club mixer.

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

Rename it to [Master]gain would be technically correct and a good idea.

Anny Idea about the effort to control the sound cards volume?

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

Renaming to "gain" and aliasing to "volume" sounds good -- I think we should leave soundcard settings alone though.

RJ Skerry-Ryan (rryan)
Changed in mixxx:
assignee: nobody → RJ Ryan (rryan)
importance: Undecided → Low
milestone: none → 1.12.0
Revision history for this message
RJ Skerry-Ryan (rryan) wrote :
Changed in mixxx:
status: New → Fix Committed
summary: - [Master],volume is a bitcursher use hadware mixer instead
+ Rename master/headphone volume to gain.
Revision history for this message
Be (be.ing) wrote :

I noticed that the control objects [Master],volume and [Master],headVolume have not been renamed. Should they be renamed and the old names deprecated and aliased to new names?

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

Thats already done, in Mixxx:
https://github.com/mixxxdj/mixxx/blob/da0f31cd220c3ad878367a5abe14044812afeb44/src/engine/enginemaster.cpp#L108

Where did you find it? Maybe we should mark them as deprecated there.

RJ Skerry-Ryan (rryan)
Changed in mixxx:
status: Fix Committed → Fix Released
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/7413

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.

Other bug subscribers

Remote bug watches

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