EmuFlight icon indicating copy to clipboard operation
EmuFlight copied to clipboard

RSSI channel not being saved - old issue, still present

Open szafranski opened this issue 4 years ago • 2 comments

Describe the bug RSSI channel is not being saved (CLI and GUI - same thing). There is some workaround (rssi_adc should be off in a first place), but it is not a proper fix I think.

To Reproduce as in this video: https://www.youtube.com/watch?v=x-qdaVpmn5o

Expected behavior RSSI channel should be saved but it is not.

Additional context It is old issue with a known workaround but it should not behave like this anyways - I think.

szafranski avatar Jan 03 '21 20:01 szafranski

yeah appears to be a gui bug rather than a firmware bug

Quick-Flash avatar Apr 02 '21 01:04 Quick-Flash

i looked at the gui code for the old issue.

src/js/tabs/receiver.js line ~258 transfers the gui value into the variable RSSI_CONFIG.channel = parseInt($('select[name="rssi_channel"]').val()); then line ~267 saves with MSP.send_message(MSPCodes.MSP_SET_RSSI_CONFIG, mspHelper.crunch(MSPCodes.MSP_SET_RSSI_CONFIG), false, save_rc_configs);

so i dont know why it fails to save.

i set a debug line for console.log, and the save executes properly irregardless of RSSI_ADC enabled or disabled.

my conclusion is that the gui performs as expected.

nerdCopter avatar Oct 08 '21 20:10 nerdCopter