-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Tripple-Osc Preset Cleanup #1099
Comments
I think it sounds terrific. IIRC, the preset data is stored inside the projects so it shouldn't break backwards compat. In terms of which branch, I would tend to lean to |
Well, I need is a green light and a branch to merge to and I'm set.
Correct! That is one of the reasons I decided to go ahead and bring this up, as I don't really have to consider how to maintain BC as it is not even relevant. |
Wouldn't it be better to throw it into master and then back port the parch
|
On 08/25/2014 08:28 AM, eagles051387 wrote:
No. |
On 08/25/2014 08:07 AM, Tres Finocchiaro wrote:
Well, a couple of things before we get ahead of ourselves here. Which presets do you want to remove and why? Is there some kind of As for adding new presets... I see no problem as long as those presets |
@diizy, I don't think he's getting ahead of himself because he's just asking where to put the pull request. Isn't the pull request the place for the granular "why is this changed?" discussion? -Tres |
On 08/25/2014 05:51 PM, Tres Finocchiaro wrote:
Well, wouldn't it be better to figure that out now so as to reduce |
While you have valid point about people wanting certain sounds, there is also the idea that users could create their own. Because of that I think the presets should be more balanced in the sounds they offer. When I say remove I am explicitly talking about presets that are extremely similar to other existing presets. A lot of the presets included are also very simple. While we do need some simple presets, I don't think that the library should be full of them because the average user would more than likely be on that level of synthesis anyways. As for my presets, I have a lot that are based off pulse width and frequency modulation. There are next to zero presets utilizing those modulations. |
I was planning on that after I dealt with the synth I'm most familiar with! Next on the list are LB302 and SID because they are the next synths I am familiar with, but I'm not opposed to working on others as well. I just want to deal with 3OSC first. |
In this case, I feel being able to see which ones removed and why would take out any guesswork or speculative questions. If I read his posts correctly, the work has already been done so we just have to determine if it's worth merging, which is hard to say until we can touch it. |
Furthermore, we've been talking about cleaning up the presets for a while and even if this merge is rejected, I think it will be a good catalyst for conversation. 🍎 |
Nope! I didn't want to start without knowing I wouldn't get sandblasted for changing presets, of all things. |
Ah... Then this certainly is the right place for it sorry for any confusion. Perhaps then to ease into this you could throw a few specific examples of improvements at the wall and see what sticks? |
Goals: Reduce the number of really simple presets. |
Sounds good! Less similar and simple presets, more different and advanced presets. I also believe StakeoutPunch has the skills and experience to add some decent presets. If we want some presets to stay, we'll just comment on the pull request. |
Sounds good. I'm going to go ahead and close this ticket. When I finish I'll submit a pull request to stable-1.1 as told, and we can discuss from there. |
Some of the 3OSC presets are redundant, not very useful, or just not the best sounding/have poor sound design.
I'm wondering if anyone would be opposed to me combing through them and tweaking/removing ones that aren't the best and adding quite a few of my own. Because I do all of my synthwork in 3OSC I have quite a few patches I'd like to see included in LMMS.
I'm set to go ahead and do this, I'm just not familiar enough with the different branches to know which one I'd need to merge to.
The text was updated successfully, but these errors were encountered: