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

Reenable AWE32 NRPN 23 and 24 #1430

Merged
merged 2 commits into from
Dec 1, 2024
Merged

Reenable AWE32 NRPN 23 and 24 #1430

merged 2 commits into from
Dec 1, 2024

Conversation

derselbst
Copy link
Member

After dealing with filters extensively and reading through the AWE32 docs yet again, I'm pretty sure that what they claim as "phase [...] filter modulation" should actually mean "filter cutoff modulation." Therefore, this PR enables AWE32 NRPN 23 and 24, causing them to behave just like the generators found in SF2.

@klerg
Copy link

klerg commented Nov 16, 2024

Interesting, here we find another case of the misleading and tricky AWE32 docs, will this result in a more accurate interpretation of the AWE32 NRPN's or what effect is this going to have on the filter overall ?

@derselbst
Copy link
Member Author

what effect is this going to have on the filter overall ?

I'm currently unable to provide you binaries, because the Windows build is broken.

@klerg
Copy link

klerg commented Nov 17, 2024

I see when will the Windows build get fixed so you can provide the binaries ?

@spessasus
Copy link
Contributor

I see when will the Windows build get fixed so you can provide the binaries ?

If you have windows, you can use WSL2 and build for Linux, then render with the -F option to a wav file. Works fine for me

Copy link

sonarcloud bot commented Nov 23, 2024

@derselbst
Copy link
Member Author

@klerg Binaries are available again. Feel free to test, I can barely hear a difference:
https://dev.azure.com/tommbrt/tommbrt/_build/results?buildId=11032&view=artifacts&pathAsName=false&type=publishedArtifacts

@klerg
Copy link

klerg commented Nov 24, 2024

I'm unable to run any of three binaries, it gives the following error:

image

Any idea how to fix this ?

@derselbst
Copy link
Member Author

@klerg
Copy link

klerg commented Nov 24, 2024

That's fine, it loads but now the first 6 seconds or so of The Nervous Filter.mid is malformed and mangled, is #1345 reverted in these binaries, or are they using #1432 smoothing ?

@derselbst
Copy link
Member Author

The smoothing logic is not included here. It will be included in the final release.

@klerg
Copy link

klerg commented Nov 26, 2024

I see so is the issue with The Nervous Filter.mid caused by #1345 being reverted and can you send binaries with smoothing before final release ?

@derselbst
Copy link
Member Author

Sorry, I was very busy the past days. I cannot postpone the release any further, so I'll merge this now. Binaries will be available in the release.

@derselbst derselbst merged commit aed6ba3 into master Dec 1, 2024
76 of 78 checks passed
@derselbst derselbst deleted the awe32-modenvfc branch December 1, 2024 13:13
@klerg
Copy link

klerg commented Dec 2, 2024

Sure that is fine and not a big deal here. I do not see a need to release a new version so soon. I just tested 2.4.1 and the 1st 7 seconds or so of The Nervous Filter.mid is still malformed and mangled so I will post it as a new issue

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

Successfully merging this pull request may close these issues.

3 participants