Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Amp Stack - parameters controlled by check boxes are still active even when it is disabled #26

Open
Gimmeapill opened this issue May 24, 2024 · 12 comments

Comments

@Gimmeapill
Copy link

Hi Hermann,

I found one minor issue while playing with the Neural Amp Modeler (which is super great by the way - thanks again!).

The Bass Boost, Presence, Reverb, LowEnd and AntiFizz are still activable with their respective check boxes even when the amp stack itself is disabled.
It is easy to disable them but ideally that shouldn't be the case, as we want a signal as dry as possible for the full-rig models.

Cheers,

LX

@Gimmeapill
Copy link
Author

Gimmeapill commented May 24, 2024

Actually, nevermind: that is quite handy once we know about it.
Adding a bit of Presence/AntiFizz etc. without having the gain stage active make sense with the models.
Maybe just adding some visual indication since the behavior can be quite puzzling?

@brummer10
Copy link
Owner

Yes, I need to remove the checkbox to disable the amp stack, as it does in fact nothing.
The main amp could only be disabled by selecting the "--" no tube option.

@Gimmeapill
Copy link
Author

Thanks, I got it now.

I ran a few tests with the latest changes, and in Guitarix standalone everything works as expected.
(It is really great to be able to remove the stuff I don't need and turn it into a "basic" NAM loader)

Now with the plugin, the UI still has the labels so it is still a bit confusing:
Screenshot_2024-05-26_16-07-04.

Also the latest changes (I think), break the restoration of the plugin state in Bitwig: "Could not read responses: End of stream".
After re-inserting the plugin instances and re-selecting the presets everything seems to be working again, but this might break things for other users...

@brummer10
Copy link
Owner

Also the latest changes (I think), break the restoration of the plugin state in Bitwig: "Could not read responses: End of stream".

mm, that shouldn't happen. I need to check what's going on there. Normally this change should be backward compatible, as it only hide widgets when needed. I may add the labels to the hide list, was just a bit lazy.

@Gimmeapill
Copy link
Author

I gave it a few more tries but I couldn't reproduce the issue again. I have now four instances opening on startup and presets are loading fine (also when loading/saving in Guitarix).
So far so good - thanks Hermann!

@brummer10
Copy link
Owner

That's good to know, thanks for testing and report back.

@kramlie
Copy link

kramlie commented Sep 13, 2024

I discovered one more bug related to this: The Master Gain is still applied even when hidden. The others have no effect though, as expected.

@brummer10
Copy link
Owner

Thanks for reporting, I need to check (and fix) that.

@brummer10
Copy link
Owner

I rechecked that and remember that's a feature, not a bug. Taking the Master Gain out of the chain could lead to unwanted volume jumps so we decided to keep it active.

@kramlie
Copy link

kramlie commented Sep 15, 2024 via email

@brummer10
Copy link
Owner

Done, The Master Gain control wont be hide any more when you switch to no tubes.

@kramlie
Copy link

kramlie commented Sep 26, 2024

Confirmed! Thanks, @brummer10!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants