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

[Discussion] Should Oversampling be saved in presets? #201

Closed
not-holar opened this issue Jun 12, 2021 · 6 comments
Closed

[Discussion] Should Oversampling be saved in presets? #201

not-holar opened this issue Jun 12, 2021 · 6 comments
Assignees
Labels
enhancement New feature or request

Comments

@not-holar
Copy link

Currently oversampling is saved in presets, but i'm not sure if it should, typically it's something you would set depending on where you use it and how powerful your machine is, not really on the settings of a preset (perhaps it matters if your preset uses a lot of gain for overdrive).

In a hypothetical situation with how it works right now, let's say you wanted to add some tape flavour when mastering, naturally, the first thing you'd do is set the highest oversampling setting, you'd change some settings around, decide you'd want to use some preset instead, start scrolling through them. Your oversampling setting would get overwritten with the (typically low) one from a preset, potentially doing so without you noticing and possibly introducing some aliasing to your master. This could be made even worse when #198 is implemented, as you wouldn't hear the problem right away.

I'm interested to hear what other people think of this.

@not-holar not-holar added the enhancement New feature or request label Jun 12, 2021
@jatinchowdhury18
Copy link
Owner

This is a good point, and something that I had thought about when originally implementing the oversampling options. The situation that you give is a good example of the problems that could arise with the current implementation. However, as a counter-example, I have a couple of presets that I typically use for vocals where I like to have more oversampling, so if I'm scrolling through presets, I would prefer for those to stay with the oversampling rate saved in the preset so that when I go to use that preset on a vocal track, I wouldn't need to also set the adjust the oversampling rate to the setting that I use with that preset.

I would definitely also love to hear more opinions on this idea.

@not-holar
Copy link
Author

As an idea, preset oversampling could act as a minimum, so the instance would follow the presets' settings as long as they're not lower than the one manually set by the user.

@jatinchowdhury18
Copy link
Owner

That's good idea... the only issue I could foresee is that the behaviour may not be immediately obvious to the user, which could lead to some confusion. I'll try to ask around today on a couple of forums and hear some more opinions.

@jatinchowdhury18
Copy link
Owner

Apologies, I've been super busy the last week or so. I'll get back on this today.

@jatinchowdhury18
Copy link
Owner

Okay, so after asking around, I think the general consensus is that the oversampling factor should be determined by the user and not affected by changing presets. Someone mentioned that there could be an option to allow presets to override the user's selected oversampling behaviour, but I worry this would be more confusing than useful in the general case.

Anyway, I'll get started on writing the code for this change this weekend.

@jatinchowdhury18
Copy link
Owner

This change is now implemented, and can be tried with the NIghtly Builds. Thanks @not-holar for the suggestion!

Thanks,
Jatin

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

No branches or pull requests

2 participants