easyeffects icon indicating copy to clipboard operation
easyeffects copied to clipboard

CLI reports wrong preset

Open sandorex opened this issue 2 weeks ago • 6 comments

EasyEffects Version

8.0.6

What package are you using?

NixOS

Distribution

NixOS 25.11

Describe the bug

Running easyeffects -s gives me following input even though i have a preset enabled, this takes up to few minutes to actually reflect the current state

Input: None
Output: None

Screenshot Image

Video (in this case it updated in few seconds) https://github.com/user-attachments/assets/5bb57225-049f-4b6d-9f6d-753feeb693b9

Expected Behavior

It should return the current active preset

Debug Log

Debug Log
Paste your log here

Additional Information

This appeared with the migration to QT

sandorex avatar Dec 09 '25 09:12 sandorex

Thanks for the report.

@wwmm We should also translate None since it's not a preset name and we generally show translated strings on the command line.

Digitalone1 avatar Dec 09 '25 12:12 Digitalone1

Do note that i do have a preset named "None" cause i did not see an option to set no preset, it's literally an empty preset, but as it is named None you cannot differentiate it from actual no preset being set which also shows as None I guess easiest solution would be to quote the name "None" and None

sandorex avatar Dec 09 '25 13:12 sandorex

Maybe we should output something as No preset set.

Digitalone1 avatar Dec 09 '25 13:12 Digitalone1

Running easyeffects -s gives me following input even though i have a preset enabled, this takes up to few minutes to actually reflect the current state

This is happening because the secondary instance that is trying to get the value is reading it from the database. But we only update the database when the autosave timout triggers it or when the window is closed.

For the update to be instantaneous we should actually make the secondary instance communicate with the local server from the primary instance and get the current value directly from it.

wwmm avatar Dec 09 '25 15:12 wwmm

Maybe we should output something as No preset set.

I think that in this case it is better to just use the empty string and show nothing in the output after the ":"

wwmm avatar Dec 09 '25 15:12 wwmm

With the functionality added by https://github.com/wwmm/easyeffects/pull/4635 it was possible to fix that. It seems fine to me now.

wwmm avatar Dec 09 '25 16:12 wwmm