diff options
author | Chris Robinson <[email protected]> | 2019-11-09 12:51:51 -0800 |
---|---|---|
committer | Chris Robinson <[email protected]> | 2019-11-09 13:04:44 -0800 |
commit | 74cbba511d6b71634d07d44a822c826400fe967d (patch) | |
tree | c36d947d53dbdfdb96a977681cd173d97876592a /utils/alsoft-config | |
parent | 89938b95cf581e8c279b187d51048dd7c1bcd069 (diff) |
Limit HRTF ambisonic decoding to second-order
The generated third-order matrix has incorrect first-order coefficients,
indicating a wonky decoder. The generated second-order matrix looks more
stable.
Diffstat (limited to 'utils/alsoft-config')
-rw-r--r-- | utils/alsoft-config/mainwindow.cpp | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/utils/alsoft-config/mainwindow.cpp b/utils/alsoft-config/mainwindow.cpp index f4f5fd45..7b1d8a91 100644 --- a/utils/alsoft-config/mainwindow.cpp +++ b/utils/alsoft-config/mainwindow.cpp @@ -129,7 +129,6 @@ static const struct NameValuePair { }, hrtfModeList[] = { { "1st Order Ambisonic", "ambi1" }, { "2nd Order Ambisonic", "ambi2" }, - { "3rd Order Ambisonic", "ambi3" }, { "Default (Full)", "" }, { "Full", "full" }, @@ -742,10 +741,13 @@ void MainWindow::loadConfig(const QString &fname) ui->enableNeonCheckBox->setChecked(!disabledCpuExts.contains("neon", Qt::CaseInsensitive)); QString hrtfmode{settings.value("hrtf-mode").toString().trimmed()}; - ui->hrtfmodeSlider->setValue(3); - ui->hrtfmodeLabel->setText(hrtfModeList[3].name); - /* The "basic" mode name is no longer supported. Use "ambi2" instead. */ - if(hrtfmode == "basic") hrtfmode = "ambi2"; + ui->hrtfmodeSlider->setValue(2); + ui->hrtfmodeLabel->setText(hrtfModeList[2].name); + /* The "basic" mode name is no longer supported, and "ambi3" is temporarily + * disabled. Use "ambi2" instead. + */ + if(hrtfmode == "basic" || hrtfmode == "ambi3") + hrtfmode = "ambi2"; for(int i = 0;hrtfModeList[i].name[0];i++) { if(hrtfmode == hrtfModeList[i].value) |