GoXLR-on-Linux / goxlr-utility

An unofficial GoXLR App replacement for Linux, Windows and MacOS
MIT License
636 stars 36 forks source link

Sampler LEDs on web interface do not match hardware #85

Closed AlienXAXS closed 1 year ago

AlienXAXS commented 1 year ago

Hi, Please accept my apologies if I am misunderstanding how this replacement software should operate, however I wanted to share a finding:

On the original GoXLR software, clearing a sample would turn off the LED on that button to clearly show that the button has nothing assigned to it - Ready for recording!

It seems goxlr-utility does not do this, even though the web interface shows that the light should be off.

I have the full sized GoXLR, running Windows 11, utility v0.12.1

Here is an image where you can clearly see the software thinks the LED is off, but the GoXLR hardware device still has it on. image

FrostyCoolSlug commented 1 year ago

Thanks for the report, it looks like something else has gone wrong either with the visualiser or the utility (none of the colours seem to match correctly).

Could you please change the log level do debug (in system -> settings), restart the utility, try clearing a sample button, then attach the log file here please?

Thanks.

FrostyCoolSlug commented 1 year ago

Just a couple of extra questions to help track this down, when you do things like mute a fader, change the FX preset and the samples presets, do the buttons correctly light up to represent that?

Finally, what firmware version are you running (can be found on System -> About).

From what I'm seeing, and what you've described, it looks like the utility is failing to apply the colour map to the device. The colours in the UI don't match the colours of the device, and 'blanking' the sampler button requires a full colour map update, which is obviously not happening despite the util thinking it has.

AlienXAXS commented 1 year ago

Hi Frosty, Here is the log file: goxlr-daemon.log

This process was:

  1. Start the app
  2. Add a sample to button A Bottom Right
  3. Clear the sample using the Clear button on the GoXLR
  4. Colours remained the same.

When you do things like mute a fader, change the FX preset and the samples presets, do the buttons correctly light up to represent that?

edit Muting of faders does change the colour, but not to the colour in the web interface but I believe this could be due to me using the colour effects where the buttons change slowly.

Changing the colours in the Lighting -> Sampler section does not reflect on the hardware at all on either A, B or C or on any three of the colour wheels.

Firmware Versions:

AlienXAXS commented 1 year ago

Thanks, does the clear button flash when you press it?

Yes

AlienXAXS commented 1 year ago

If debugging is easier: I am more than happy to jump on a discord call and go through steps, just ask :)

FrostyCoolSlug commented 1 year ago

Just saw the edit, thanks.

So definitely something wrong with the colour map, I'll do some checking.

FrostyCoolSlug commented 1 year ago

I believe this could be due to me using the colour effects where the buttons change slowly.

Just to clarify, under the official app, were you using the global animation feature?

AlienXAXS commented 1 year ago

Yes bud. I'm just doing testing now to see if the problem persists if i disable this feature then reload that disabled profile on GoXLR Utility.

FrostyCoolSlug commented 1 year ago

Ok, that'll probably do it, the util doesn't support the animations, so the GoXLR might be 'stuck' with them on as the util can't turn them off..

Can you simply unplug your GoXLR from Power, then plug it back in again and see if the util picks the colours up correctly?

AlienXAXS commented 1 year ago

Hi, It seems that loading a no effect profile did the trick, also tried replugging power and that also worked as intended too.

Sorry to have been a pain here, i was not aware the colour animations didnt work.

FrostyCoolSlug commented 1 year ago

No problem at all, that's on me. I'll make a small adjustment to the code to force animations off on profile load to ensure this doesn't trip anyone else up.

AlienXAXS commented 1 year ago

Sounds perfect, thanks for making the app and the great support :)

FrostyCoolSlug commented 1 year ago

This will be fixed for the 1.0 release (7eeb56c), If I get time, I'll see if I can actually support animations, but for now this will suffice.

Thanks again for the report!