-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
[BUGFIX] Fixed Polymod and Mods Support in FlxPartialSound #2724
[BUGFIX] Fixed Polymod and Mods Support in FlxPartialSound #2724
Conversation
I've tested this and seems to function with base game songs normally and good! I'll give shout to @EliteMasterEric who has more brain on polymod stuff! I have a branch in our funkin-secret repo eric if u wanna take a peak at it ! |
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.
Tested with a mod and it seems to work !
a.mp4
This wasn't working for me, and I struggled to debug it until I realized I was on a ZIP mod and apparently Polymod wasn't implementing the override for loadBytes correctly. |
Silly Eric
…On Mon, Jun 17, 2024, 12:24 a.m. Eric ***@***.***> wrote:
This wasn't working for me, and I struggled to debug it until I realized I
was on a ZIP mod and apparently Polymod wasn't implementing the override
for loadBytes correctly.
—
Reply to this email directly, view it on GitHub
<#2724 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ATNYONPCLBJT2C6GDRSXLHLZHZQIZAVCNFSM6AAAAABJAYD7M6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZSGE4TCMBTGU>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
figured out the issue
i'll get it fixed soon using |
I've just pushed a new commit to the pr on the FlxPartialSound repo that fixes this Here's a little video of it working on base, folder mods and zip mods songs 2024-06-17.15-21-02.mp4 |
This should be merged with the changes in this pr on FlxPartialSound