-
-
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
fixing "Alf42red-Mauiwowi" project file (by swapping it with version from 0.4x) #6463
fixing "Alf42red-Mauiwowi" project file (by swapping it with version from 0.4x) #6463
Conversation
🤖 Hey, I'm @LmmsBot from github.com/lmms/bot and I made downloads for this pull request, click me to make them magically appear! 🎩
Windows
🤖{"platform_name_to_artifacts": {"Windows": [{"artifact": {"title": {"title": "32-bit", "platform_name": "Windows"}, "link": {"link": "https://ci.appveyor.com/api/buildjobs/o559ub7ibrks8qar/artifacts/build/lmms-1.3.0-alpha-msvc2017-win32.exe"}}, "build_link": "https://ci.appveyor.com/project/Lukas-W/lmms/builds/44165672"}, {"artifact": {"title": {"title": "64-bit", "platform_name": "Windows"}, "link": {"link": "https://ci.appveyor.com/api/buildjobs/m6rqm7qsvx3upe11/artifacts/build/lmms-1.3.0-alpha-msvc2017-win64.exe"}}, "build_link": "https://ci.appveyor.com/project/Lukas-W/lmms/builds/44165672"}]}, "commit_sha": "af90a89d686ea3064f061c406a47b556ffe414c5"} |
Issue and fix confirmed. I'm curious about when this happened and why. |
Probably some old/odd upgrade path that we've since fixed. |
@kamkamkamkamkamkamkamkam Extra points for removing the unused channels on the mixer (right click any mixer channel -> Remove unused channels). |
@Monospace-V |
Also, is there a reason the checks are failing now suddenly? |
This seem to be related to the merging of #6174 .
You say in the description above that the bugg affects ALL instances of zasfx. This isn't the case for me and doesn't seem to be the case for @Monospace-V . In my case it's only the first and third track that are silent. It seem to be related to switching the ladspa effects to submodules. In this case the plugin Calf Filter LADSPA. |
In my case, the first and third track make sound, but very silent. If I amplify it over a few times it becomes audible. @zonkmachine do see if this happens to be the case for you too? OP, I don't quite understand what you mean when you say my screenshots suggest otherwise, can you clarify? I've solo'ed the ZASFX tracks, since those seem to be the ones causing problems, so I've isolated. |
Yes indeed! If I cram up the volume of my modest headset there is sound. There could also be an issue related to the LFO Controllers. The calf filters are controlled in the same way and if you turn the BASE knob there is once more sound. This time from opening up the filters. If I right click the Frequency knob on the filters and choose Set linear the issue is fixed. |
Did it work this way in previous versions of LMMS too? If yes, then this may have been how it was intended to sound. What version of LMMS is being tested with? 0.4 and 1.1, i can't see this project in demo files. |
#3940 was essentially a direct clone of 0.60 and a partial rename (from #3987 was much more involved, was a full backport of the LADSPAv1 interface to the 0.90 Calf codebase, lead largely by @JohannesLorenz. If this PR is the cause, the upgrade mappings are here if someone can help out: https://github.com/LMMS/lmms/pull/3987/files#diff-5c8fb43716d5630ce8bc7b97df7bd34fa05d92762129de919fd84e0670c27f7d. |
Testing the project and fix further. Pad below means calf filter on mixer channel 3 and guitar means calf filter on the guitar track (track 3). What differs between versions of lmms-1.2.2 and lmms on master (f39b3d5) is whether the filter cutoff knob is linear or logarithmic and if that same knob is connected to a controller or not. This fix seem to be more complete applied to lmms-1.2.2 than to master. You need to also set the cutoff to linear and connect them to the controllers 1.2.2 fixed demo master fixed demo |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See: #6463 (comment)
There seem to be more going on here and quite possibly other projects could be affected. What's missing now is controller connections on the filters and linear response instead of logarithmic. This should maybe be fixed by an upgrade routine instead of a manual fix.
As you see from my reply above it seem to be a bit more complicated.
I've tested on lmms-1.2.2 and on master with the file provided in this fix and also on newer, lmms-1,1.3 which also fixes this, at least on stable-1.2.2. I think the project degraded in at least two steps somewhere after lmms-1.1.3 . |
I think this right here is the actual problem. When you open both instances of Calf Filter and set the "Frequency" knob to "Linear" the project then sounds exactly identical to how it's supposed to sound.
Actually, this is probably a better idea 😅 |
For a very long time, the version of "Alf42red-Mauiwowi" bundled with LMMS has been completely broken with all instances of ZynAddSubFX being silent for some unknown reason (despite the project file growing in size somehow). This has been a thing since I think 1.1x but it could've started in 1.2x. I replaced the project with a version from 0.4.15 (which was last modified in 0.4.8) and it plays fine in the latest build of LMMS
For reference, here's how the song currently sounds in LMMS.
Here's how it's supposed to sound.