YesNoOk
avatar

xnaMugen Build 22343 (1/29) (Read 27111 times)

Started by FrantzX, January 30, 2009, 08:53:59 am
xnaMugen Build 22343 (1/29)
#1  January 30, 2009, 08:53:59 am
  • avatar
  • **
    • code.google.com/p/xnamugen/
Get it here!

More random bug fixes, mostly around Jesuszilla's Felicia.

Note: For the time being xnaMugen will read & use correctly the incorrect trigger syntax of "AnimElem >= number". In later releases, this will be controlled by a setting in the xnaMugen.ini file.
Last Edit: January 30, 2009, 09:01:56 am by FrantzX
Re: xnaMugen Build 22343 (1/29)
#2  January 30, 2009, 09:35:10 am
  • ****
I can't say much about what's going wrong with the coding, but have you tried Felicia using the A groove? She quickly causes the engine to lag more and more with every new animation activated untill the screen is stagnant.

Just in case it matter Player 2 was using P groove.
Re: xnaMugen Build 22343 (1/29)
#3  January 30, 2009, 10:24:53 am
  • avatar
  • **
    • code.google.com/p/xnamugen/
OK, I have been able to reproduce the issue. It is caused by something in Felicia's code that is creating Explods, nonstop. I'll look into it some more. I don't know yet weather it is a xnaMugen bug or a Felicia bug yet.

Edit:
Getting closer to the problem. It only happens when P2 is Felicia in P groove. That would explain why I never saw it before. I do most of my testing with P1 & with P2 as Kung Fu Man.
Last Edit: January 30, 2009, 10:56:25 am by FrantzX
Re: xnaMugen Build 22343 (1/29)
#4  January 30, 2009, 11:33:25 am
  • avatar
  • **
    • code.google.com/p/xnamugen/
I found the bug. It is caused by Jesuszilla's Felicia.
Here is the cause. The first statecontroller is good but the second one is what causes the problem. Notice the difference in the triggers:

[State 6120, Explod];BAR1P
type = Explod
trigger1 = !NumExplod(6120)
trigger1 = teamside = 1
anim = 6207
ID = 6120
postype = Left
pos = 140,230
ownpal = 1
bindtime = -1
ignorehitpause = 1
ontop = 1
facing = 1
pausemovetime = -1
supermovetime = -1
scale = .5,.5

[State 6120, Explod];BAR2P
type = Explod
trigger1 = !NumExplod(6120)
trigger4 = teamside = 2
anim = 6208
ID = 6120
postype = Right
pos = -140,230
ownpal = 1
bindtime = -1
ignorehitpause = 1
ontop = 1
facing = 1
pausemovetime = -1
supermovetime = -1
scale = .5,.5

The reason that this bug does not show up in WinMugen is because WinMugen has a limit of the number of explods that can exist at one point in time. xnaMugen has no such limit, so every tick creates a new explod.
Re: xnaMugen Build 22343 (1/29)
#5  January 30, 2009, 11:34:57 am
  • avatar
  • **
    • code.google.com/p/xnamugen/
I think my next goal for xnaMugen is a better clipboard/diagnostic window to make it easier to see when things like this happen.
Re: xnaMugen Build 22343 (1/29)
#6  January 30, 2009, 11:41:06 am
  • ******
  • Legendary XIII
  • I am the eye of the storm to come!
    • New Zealand
    • network.mugenguild.com/cyanide/
Quote
[State 6120, Explod];BAR2P
type = Explod
trigger1 = !NumExplod(6120)
trigger4 = teamside = 2
anim = 6208
ID = 6120
postype = Right
pos = -140,230
ownpal = 1
bindtime = -1
ignorehitpause = 1
ontop = 1
facing = 1
pausemovetime = -1
supermovetime = -1
scale = .5,.5
trigger1 then straight to trigger4? In my experience that breaks. Is the problem solved if it's changed to trigger1 instead?

Edit: Cool, my afterimages look mostly right now.


In M.U.G.E.N there is no magic button

They say a little knowledge is a dangerous thing, but it's not one half so bad as a lot of ignorance.
Last Edit: January 30, 2009, 11:48:35 am by Cyanide
Re: xnaMugen Build 22343 (1/29)
#7  January 30, 2009, 11:51:40 am
  • avatar
  • **
    • code.google.com/p/xnamugen/
Yes, it fixes the problem.

xnaMugen allows for "triggern" where n is any integer number. (Note to self: Add check to make sure n is positive.)
xnaMugen will check the triggers in numerical order. If a number is missing, it simply goes to the next. Is this incorrect behavior?
Re: xnaMugen Build 22343 (1/29)
#8  January 30, 2009, 12:00:36 pm
  • ***
  • THATS MR HASEO TO YOU
    • haseo.ucoz.com/
in mugen it simply stops if the next trigger isnt the next number
for example

this works

trigger1 = whatever
trigger2 = whatever2
trigger3 = whatever3

and this doesnt

trigger1 = whatever
trigger3 = whatever3

mugen completely skips trigger3
Re: xnaMugen Build 22343 (1/29)
#9  January 30, 2009, 06:08:59 pm
  • ******
  • [E]
    • Mexico
It is incorrect behaviour, but I think it's more desirable that way; so we can organize our triggers better.
Re: xnaMugen Build 22343 (1/29)
#10  January 30, 2009, 09:35:31 pm
  • ******
  • Legendary XIII
  • I am the eye of the storm to come!
    • New Zealand
    • network.mugenguild.com/cyanide/
How is being able to enter any positive number better? should be 1-2-3 etc. 1-8-22 makes no sense.


In M.U.G.E.N there is no magic button

They say a little knowledge is a dangerous thing, but it's not one half so bad as a lot of ignorance.
Re: xnaMugen Build 22343 (1/29)
#11  January 30, 2009, 10:30:27 pm
  • ******
  • [E]
    • Mexico
I was expecting that.

let's say you have

type=changestate
value = special
trigger1= ctrl
trigger2= combo from weak punch
trigger3 = combo from weak kick

if you disallow it to combo from weak punch you have to

type=changestate
value = special
trigger1= ctrl
;trigger2= combo from weak punch
trigger2 = combo from weak kick


while

type=changestate
value = special
trigger1= ctrl
;trigger2= combo from weak punch
trigger3 = combo from weak kick

is easier to read and edit.

Re: xnaMugen Build 22343 (1/29)
#12  January 30, 2009, 11:33:22 pm
  • ******
  • Legendary XIII
  • I am the eye of the storm to come!
    • New Zealand
    • network.mugenguild.com/cyanide/
I disagree with that. Allows too much sloppy coding. It might work. But if people stop bothering to label their triggers correctly it becomes more difficult to read in the end. A suffix to the trigger line that says ignore this but still confirm trigger2 exists would be better than any positive number that's the next highest.


In M.U.G.E.N there is no magic button

They say a little knowledge is a dangerous thing, but it's not one half so bad as a lot of ignorance.
Re: xnaMugen Build 22343 (1/29)
#13  January 30, 2009, 11:36:47 pm
  • ******
  • [E]
    • Mexico
I don't really see what's sloppy on that, since the whole concept of numbering triggers is pretty "blah" ideally they should be named.
Re: xnaMugen Build 22343 (1/29)
#14  January 30, 2009, 11:38:04 pm
  • ******
  • Loyal to the Game
    • USA
    • http://jesuszilla.trinitymugen.net/
Oh shit. :omg:


Fixed. At this point, I definitely need to make another update soon. :P
Last Edit: January 30, 2009, 11:42:01 pm by Jesuszilla
Re: xnaMugen Build 22343 (1/29)
#15  January 30, 2009, 11:44:40 pm
  • ******
  • Limited time to use Infinite power !
    • France
    • network.mugenguild.com/cybaster/
Let's rename this "xnaMugen" section into "Let's debug JZ's Felicia with a good parser". ;P
Re: xnaMugen Build 22343 (1/29)
#16  January 30, 2009, 11:53:14 pm
  • ******
  • [E]
    • Mexico
Felicia should be included with xnamugen instead of kungfuman.
Re: xnaMugen Build 22343 (1/29)
#17  January 31, 2009, 01:11:43 am
  • ******
  • Loyal to the Game
    • USA
    • http://jesuszilla.trinitymugen.net/
OK, tested (with a fixed version of Felicia, which I ought to get to you).

General
  • I still have the black on the scrolling background in the main menus whenever I use the old shader, and I still can't see anything but fonts with the new shader.
  • Characters still slide when they go from walk to crouch even if they're coded to not do that >_<.
  • After using a crouching basic and still holding down, the character goes to the stand/transition rather than directly to the crouch state.


Felicia
  • Can't chain or cancel like I should, and I can still go into the Rolling Buckler follow-ups (Rolling Upper, Neko Punch, Slide Kick) too quickly.
  • Please Help Me! doesn't work at all.
  • Second hit of the uppercut in the Lvl3 & MAX versions of Dancing Flash doesn't connect.
  • The land spark in the custom land state (1305) is misaligned.
  • Groove Point display is misaligned and has incorrect PalFX whenever Felicia gets hit.
  • Power level indicators are misaligned in C-Gauge.
  • Display messages are misaligned.
  • Why does she seem to shift positions in her intros?
  • Hitsparks seem to flicker near the end of their animation. Dunno what exactly is going on, though.
  • Ground hit sound doesn't play when I KO with Dancing Flash (Felicia vs. Felicia, although the problem could exist with other characters as well).
  • Negative Edge commands (holding button, motion, then releasing button in order to use a move) don't work. To give an example of a negative edge command, QCF+P. Instead of doing all that in order, Negative Edge allows me to hold P, then do QCF, then release P. The move should come out.


Also, tested my Andy on it. The last hit of his Shoryu-Dan won't knock the opponent down, and something weird happened whenever I used Bakushin on Vans's Ralf... Whenever he used LP, the super spark sound would play. With HP (I think), it looked like he was trying to throw the enemy. Then it crashed after Ralf tried that a few more times.
Spoiler, click to toggle visibilty

I wouldn't really focus on Andy too much ATM though. I didn't really intend to report any problems he had, but then this happened. :-X
Last Edit: January 31, 2009, 01:31:28 am by Jesuszilla
DOUBLE POST
#18  February 01, 2009, 04:30:38 am
  • ******
  • Loyal to the Game
    • USA
    • http://jesuszilla.trinitymugen.net/
Almost forgot, if one uses LP and the other HP, then HP should win. Right now, both Felicias get hit. I suppose you haven't implemented priority? Because my Felicia has a higher priority for heavy than she does light.
Re: xnaMugen Build 22343 (1/29)
#19  February 03, 2009, 08:14:37 am
  • avatar
  • **
    • code.google.com/p/xnamugen/
While you are fixing Felicia, please look into this as well.

Spoiler, click to toggle visibilty

I would recommend running you other characters thru xnaMugen so the parser can check them as well.
Re: xnaMugen Build 22343 (1/29)
#20  February 04, 2009, 12:11:28 am
  • ******
  • Loyal to the Game
    • USA
    • http://jesuszilla.trinitymugen.net/
Done. Will check Andy to see if he has that problem as well. All done and the updates are up!
Last Edit: February 07, 2009, 03:45:13 am by Jesuszilla
Re: xnaMugen Build 22343 (1/29)
#21  February 07, 2009, 09:29:02 am
  • avatar
  • **
    • code.google.com/p/xnamugen/
These are the state controllers for the Rolling Buckler follow-ups:
[State 1000, ChangeState]
type = ChangeState
triggerall = Time > 4
trigger1 = Command = "x"
trigger2 = Var(50) = 1 || Var(50) = 2;NumTarget(5)
trigger2 = Command = "x"
value = 1025
ignorehitpause = 1
ctrl = 0

[State 1000, ChangeState]
type = ChangeState
triggerall = Time > 4
trigger1 = Command = "y"
trigger2 = Var(50) = 1 || Var(50) = 2;NumTarget(5)
trigger2 = Command = "y"
value = 1015
ignorehitpause = 1
ctrl = 0

[State 1000, ChangeState]
type = ChangeState
triggerall = Time > 4
trigger1 = Command = "z"
trigger2 = Var(50) = 1 || Var(50) = 2;NumTarget(5)
trigger2 = Command = "z"
value = 1005
ignorehitpause = 1
ctrl = 0

[State 1000, ChangeState]
type = ChangeState
triggerall = Time > 4
trigger1 = Command = "a"
trigger2 = Command = "b"
trigger3 = Command = "c"
trigger4 = Var(50) = 1 || Var(50) = 2;NumTarget(5)
trigger4 = Command = "a"  || Command = "b" || Command = "c"
value = 1035
ignorehitpause = 1
ctrl = 0

It looks to me that you should be able to transition to a follow up by pressing the correct button 5 ticks into the Rolling Buckler's state. Of course, the trigger "Var(50) = 1 || Var(50) = 2" makes it look as it you may have another coding issue.

---

This is in the command file. You are missing trigger3.
[State -1, Chain]
type = CtrlSet
triggerall = !Ctrl
trigger1 =  (StateNo = 52 || StateNo = 101 || StateNo = 106 || StateNo = 5120 || StateNo = 1305) && AnimTime = 0 ;Land States
trigger2 =  (StateNo = [200,499]) && AnimTime = 0 ;Standing and Crouching Basics
trigger4 = ((StateNo = 5201) || (StateNo = [900,999])) && AnimTime = 0 ;System States
trigger5 =   StateNo = 2800 && AnimTime = 0       ;Counter Attacks
trigger6 =  (StateNo = 5001 || StateNo = 5011 || StateNo = 151 || StateNo = 153) && HitOver
value= 1
Re: xnaMugen Build 22343 (1/29)
New #22  February 07, 2009, 09:36:52 am
  • ******
  • Loyal to the Game
    • USA
    • http://jesuszilla.trinitymugen.net/
Now that I look at the follow-up code, it does seem redundant... Removed.


Also thanks. Can't believe I missed that last one.





Later today I will do a quick update to the main archive and tell those who already downloaded it how to fix it. Updates uploaded. Gonna sleep for now.
Last Edit: February 07, 2009, 10:06:19 am by Jesuszilla