-
-
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
Almost all presets has no envelopes #5527
Comments
This is a job for me. Can I do it? |
@dj-pixus Wooo! |
I fixed the problematic presets of builtin instruments except for LB302, as it does not support envelopes. |
If only that was it, but each individual sample saved as a AFP-preset has its own hardcoded Envelope. It is simple XML, so perhaps some 'sanity' basic std could be bulk edited in, through some script, as a minimum 'solution', but it would not result in a flawless collection of samples. |
What a timing! Just added the newly repaired presets to the PR, then read your comment. :) |
The sustain, i mean the lowest value is 0, highest is 1 at most instruments. But in BitInvader, 0 is 1 and 1 is 0. Not a problem with the sounds itself, but a technical inconsistency that made me recheck everything 6 times. |
👼 to that |
i did some changes to initial presets that could give a better audio experience. it is a mmpz compressed. |
@Clutterbyte The issue is already coded in a PR -Have you made code-changes, or are your changes made inside lmms? |
There were changes only inside lmms project, not in code. Also clicking was more to attack with value 0.0, not to hold have value 1.0 |
|
@Clutterbyte Oki, but there is an issue with your idea, or perhaps a better expression would be: Your idea is good, but it must be done differently. |
Envelope is saved in presets themselves, and even presets are hard-coded in projects. So why would a default setting break compatibility, I ask? |
@musikBear ok, so envelope cannot start active and with a diferent configuration in every generator, but as i understand, is posible to change another parameters in vanilla instruments like organic volume from 100 to 50%, watsyn oscs to -12db, etc? |
@dj-pixus
Because non of the AFP-presets has any envelope at all! @Clutterbyte Yes it a parameter IS changed, then it is written in the file. The issue with all Samples (AFP-presets are that nothing has been changed! Browse trough them in mySamples all has no own envelopes and all looks like the current default: |
@musikBear ive an idea: make that lmms recognize old projects (created in previous version) and automatically for no break projects with AFP, disable envelope if it is off. this zip contains modifications in original lmms templates getting more natural sounds in these kits (adding ladspa reverb). Example: |
It has been suggested many times :p
Sorry No! That we can not use! Those instruments would technically be presets, and then they belong in the folder My-presets. |
No problem! i would known if The 'solution' of "sane envelopes for every AFP-preset" will come before budislav´s ui change, or it isnt planned in a near future. |
I think that is far away. 2.0 or .. -Not sure its being worked atm |
I want to target single window UI for 2.0. Currently the only work on it is the #5261. There's a lot of core work that needs to go into it though. |
Sound great -One thing to at least consider could be ability to scale all text -The bat 🦇 whispered :p |
Bug Summary
I have searched, but no issue seams to address this, but Almost all presets in LMMS has no envelopes. The zasfx presets does, but they are of cause internal, and not related to LMMS own ENV/LFO tab.
Most problematic are the samples, where many clicks and pops, but i have not been through all preset, so i cant say how many we got with artefacts.
Argumentation
On youTube LMMS is ridiculed and dissed as being of low quality. Popping and clicking presets is part of this. It is reasonably easy to fix. No coding is needed, just edit and resave.
Any user with good understanding of the presets, can do this.
Steps to reproduce
Open any default preset
Expected behavior
LMMS presets ought to have an envelope, that prevents clicks and pops from attack ao release. A default All-pass-filter could also be set.
This would not inflict old projects or any saved presets. All presets has envelopes hardcoded in both project-file, and preset-files.
Actual behavior
Most presets does not have either envelope or default filter
Affected LMMS versions
All
Suggested method
All presets are renamed with an _ infront of current name with a batch-script, then they can be saved with new envelopes, without the leading underscore, and nothing would have changed. Total backward comparability is preserved.
The text was updated successfully, but these errors were encountered: