ikemen-engine / Ikemen-GO

An open-source fighting game engine that supports MUGEN resources.
https://ikemen-engine.github.io
Other
668 stars 118 forks source link

Explod's vel and accel ignore bindtime. #1880

Open Kuroteba opened 1 week ago

Kuroteba commented 1 week ago

Describe the bug

Explod's vel and accel ignore bindtime. In MUGEN, the values of vel and accel are ignored while bindtime is enabled.

To Reproduce

1.Download The Animated Colonel (remake version). 2.The Animated Colonel's Breakthrough hits the ground running.

Expected behavior

The left and right Colonel displays are different from MUGEN.

Screenshots / Video

https://github.com/ikemen-engine/Ikemen-GO/assets/99640380/01ece509-21cf-4d2f-9588-b2b3f0a039a4

Engine Version (or source code date)

v0.99.0 and nightly

Operating system

Windows

Extra context or search terms

No response

Wftk commented 1 week ago

Which version of mugen you using? Because in 1.1b it behaves same as in Ikemen GO

https://github.com/ikemen-engine/Ikemen-GO/assets/132805988/88d42824-3c06-4f20-9fcc-5154eef7274e

Wftk commented 1 week ago

Upd: Tested on 1.0 and indeed it doesnt happen there. So seems like its 1.0 compatability issue.

potsmugen commented 1 week ago

The char's mugenversion might not be correct, too. Can you upload or link the char for testing?

Kuroteba commented 1 week ago

The MUGEN used in the video is WinMUGEN.

waspennator commented 1 week ago

The char's mugenversion might not be correct, too. Can you upload or link the char for testing?

https://drive.google.com/file/d/1S9sEATEueHRu1YTsPKGcpzzELjcSq86n/view Download link for the character I believe can be found here, I can also confirm the same thing happens on Ikemen GO and on Mugen 1.1b. The character version is set as "MugenVersion= 04,14,2002 ;Mugen version of the character",

potsmugen commented 1 week ago

Ah so it's a case of Ikemen doing it like Mugen 1.1 did for WinMugen chars instead of doing it like WinMugen itself did. Minor issue I think. Mugen 1.1 level of compatibility has more or less been the goal I think.

K4thos commented 1 week ago

We're not aiming for better backward compatibility than Mugen itself. If someone is willing to add a special compatibility layer for this, go for it, but the current behavior is not considered a bug.