Author Topic: New Settings entry: Default Audio Volume  (Read 2701 times)

Offline johnty

  • Newbie
  • *
  • Posts: 46
    • View Profile
New Settings entry: Default Audio Volume
« on: April 30, 2019, 11:32:19 PM »
I usually play sequences with the Audio set to Very Quiet since I’ve heard the music for the effect dozens of times.  When I want to hear it at full production volume I switch to Loud.  When I open xLights at midnight and forget to change Audio from Loud to Very Quiet I’m blasted with music.  Last year I asked if the last Audio setting could be saved when exiting xLights.  Keith said no, the Very Quiet setting was primarily for the Zoom call.  Gil correctly pointed out that I could open Windows Volume Mixer and set xLights’ volume at a lower level.  True enough but then I have no headroom to make it louder without clickety-clickety back into the Volume Mixer.

So, how about an entry in Settings for Default Audio Volume?  Default would be Loud so the addition wouldn’t affect anybody unless they wanted to take advantage of it.  I think it would look like this in the drop-down Settings menu:

Default Audio Volume               >      •  Loud

                                                            Medium

                                                            Quiet

                                                            Very Quiet

Programming should be simple.  At start-up, read the Default Audio Volume setting, set the switch in the corresponding Audio setting, done.  This method has the added advantage that it is persistent across a crash in xLights.  Should work fine, last a long time.

Please?


Offline johnty

  • Newbie
  • *
  • Posts: 46
    • View Profile
Re: New Settings entry: Default Audio Volume
« Reply #1 on: May 11, 2019, 11:57:05 AM »
The tacit response is overwhelming.

Offline ozaz

  • Full Member
  • ***
  • Posts: 188
    • View Profile
Re: New Settings entry: Default Audio Volume
« Reply #2 on: May 12, 2019, 08:52:45 PM »
if it makes you feel better johnty, i'd like that enhancement too
I know just enough to be dangerous :0

Offline Gilrock

  • Supporting Member
  • Hero Member
  • *
  • Posts: 6946
    • View Profile
Re: New Settings entry: Default Audio Volume
« Reply #3 on: May 13, 2019, 07:30:31 AM »
The tacit response is overwhelming.

What do you prefer.  I can give a smart remark when I don't like an idea or I can say nothing.

Offline johnty

  • Newbie
  • *
  • Posts: 46
    • View Profile
Re: New Settings entry: Default Audio Volume
« Reply #4 on: May 17, 2019, 09:06:08 PM »
The tacit response is overwhelming.

What do you prefer.  I can give a smart remark when I don't like an idea or I can say nothing.

A smart remark which is your forte with perhaps a few words of why it's a bad idea.

Offline Gilrock

  • Supporting Member
  • Hero Member
  • *
  • Posts: 6946
    • View Profile
Re: New Settings entry: Default Audio Volume
« Reply #5 on: May 18, 2019, 08:28:27 AM »
You said "Keith said no" and then you said "Programming should be simple".  You pretty much guaranteed it won't get implemented with that second part...lol.

Offline TypingOffKey

  • Newbie
  • *
  • Posts: 13
    • View Profile
Re: New Settings entry: Default Audio Volume
« Reply #6 on: May 20, 2019, 01:14:21 PM »
I took a stab at implementing this feature, and my pull request (the code changes I proposed, for those who aren't familiar with github lingo) was accepted and merged, so you should see this topic addressed in the next release. There won't be a separate menu selection for default audio volume, but whatever you set in the existing "Audio" menu will be saved and restored at the next launch of the program.
Always willing to get together / help fellow lighting nuts in the Wichita, KS area.
Light-O-Rama user since 2007
xLights user since 2014
Showing off: https://youtu.be/H8K5gHBPwpU

Offline Gilrock

  • Supporting Member
  • Hero Member
  • *
  • Posts: 6946
    • View Profile
Re: New Settings entry: Default Audio Volume
« Reply #7 on: May 20, 2019, 01:26:22 PM »
Yep slowing the xlights program down 1 useless feature at a time.

Offline johnty

  • Newbie
  • *
  • Posts: 46
    • View Profile
Re: New Settings entry: Default Audio Volume
« Reply #8 on: May 20, 2019, 01:54:45 PM »
I took a stab at implementing this feature, and my pull request (the code changes I proposed, for those who aren't familiar with github lingo) was accepted and merged, so you should see this topic addressed in the next release. There won't be a separate menu selection for default audio volume, but whatever you set in the existing "Audio" menu will be saved and restored at the next launch of the program.

Even better.  Thanks TypingOffKey!

Offline babybear

  • Hero Member
  • *****
  • Posts: 540
  • Here to make everyones show better
    • View Profile
Re: New Settings entry: Default Audio Volume
« Reply #9 on: May 20, 2019, 06:39:15 PM »
So, what you are saying is that anyone, at anytime, can add anything to Xlights, good or bad. Is that not going to break Xlights.  I see a mess coming.
JimmyG
Rochester, New York

Offline aknflyer

  • Jr. Member
  • **
  • Posts: 76
  • 43 year Telecom Tech
    • View Profile
Re: New Settings entry: Default Audio Volume
« Reply #10 on: May 20, 2019, 11:52:30 PM »
That is problematic, one user who cannot work the volume control, slows it for all of us?

Offline Gilrock

  • Supporting Member
  • Hero Member
  • *
  • Posts: 6946
    • View Profile
Re: New Settings entry: Default Audio Volume
« Reply #11 on: May 21, 2019, 08:06:43 AM »
No not just anyone can add code.  Anyone can create an account and modify code and do a Pull Request and then any developer with main branch access can merge the code in.  Keith did the merge for this.  This change by itself is probably a negligible change in speed but whenever any logic is added it does take more CPU cycles to process the logic.  Just think about when users wanted the effects to snap to the timing marks.  Well that required code that compared the position of the edge of the effect with all the active timing marks every single time the effect changed position.  You add enough of those features and pretty soon people are complaining the GUI is sluggish or with a change like this its startup and shutdown time that gets affected.  You see how long it takes xLights to get running when you launch.  Also recently I noticed a change in lyric track breakdown speed.  When I first wrote that code it took about 15 seconds to import the dictionaries the first time you do a breakdown.  Now I'm not sure what was changed but there is a pretty dialog to show the progress of the import so now you can see how much longer you need to wait.  Problem is now it's like 3 times slower.

Offline aknflyer

  • Jr. Member
  • **
  • Posts: 76
  • 43 year Telecom Tech
    • View Profile
Re: New Settings entry: Default Audio Volume
« Reply #12 on: May 22, 2019, 12:34:58 AM »
I would like lean and fast, and forget about the addons that would take a couple extra clicksto make work. My show is small, but for a large show user it could probably add up. Just my 2 cents.