Action.repeat() won't repeat.

Kamozo

Hello, why won't "repmo" repeat itself?
I would like the action to continuously run after "phase_1", yet it only runs once. How do I fix this? Thank you.

        up = A.move_to(self.size.w/2, 350, 9,TIMING_EASE_BACK_OUT)	
		mo = A.move_to(random.randrange(100,924),random.randrange(300,700), 2, TIMING_EASE_BACK_IN_OUT)
		repmo = A.repeat(mo, -1)
		phase_1 = A.sequence(stationary, up) #stationary appears earlier
		self.boss.run_action(A.sequence(phase_1, repmo))

This is only a snippet of code, I'm just highlighting the bits that need looking at. For example, although the code doesn't specify, A = Action.

omz

The action does repeat, but the random coordinates are only calculated once, so each repetition moves to the same point (which has the same effect as if it wouldn't repeat at all).

If you want new random coordinates with every repetition, you'd need to use a custom call action.

Kamozo

Ahh, I understand that, thanks.
I'll have to do some more reading now, because the call function confuses me, it doesn't take much code does it? Could you give an example of how I would do it, if that isn't too much to ask?
But anyway, thanks, I'll look into it.

omz

Something like this should work:

up = A.move_to(self.size.w/2, 350, 9, TIMING_EASE_BACK_OUT)
d = 2.0 # duration
# Inline function for the call action:
def move_random():
  x, y = random.randrange(100,924), random.randrange(300,700)
  self.boss.run_action(A.move_to(x, y, d))
# The call action doesn't wait until the move action finishes, so add a wait action for that:
repmo = A.repeat(A.sequence(A.call(move_random), A.wait(d)), -1))
phase_1 = A.sequence(stationary, up)
mo = A.move_to(, 2, TIMING_EASE_BACK_IN_OUT)
self.boss.run_action(A.sequence(phase_1, repmo))

(I actually used a slightly simpler example for testing, so there might be some typos here)

Kamozo

Wow, thank you, that helps me a lot.

Kamozo

This post is deleted!

omz

Sorry, there's a closing parenthesis too much, it should be:

repmo = A.repeat(A.sequence(A.call(move_random), A.wait(d)), -1)

Kamozo

Oh yeah, I'm not sure how I didn't notice that.
Where does the code you've provided fit into mine?

Kamozo

This post is deleted!

Kamozo

Nevermind, I've figured it.
Thanks for all your help.

Webmaster4o

@omz said:

Sorry, there's a closing parenthesis too much, it should be:

It sounds much less awkward to an American if you say "there's one too many closing parentheses," but not a big deal. Just putting it out there

chriswilson

I hadn't realised the A.call() thing existed - loads of possibilities for animations!