your minimum frame is 39?
frames is unknown

onout"set(frame,1);"

I think your inc max and min are mixed up and don't even make sense if you don't want it to repeat.. the documentation says it loops using that.
inc(variable,byvalue*,max*,min*)
dec(variable,byvalue*,min*,max*)

<action name="animate">
ifnot (frame == 40,inc(frame,1));
mul(ypos,frame,frameheight);
txtadd(crop,'0|',get(ypos),'|',get(framewidth),'|',get(frameheight));
delayedcall(0.06, animate() );
</action>

<action name="animate">
ifnot (delaykilled ==true,
ifnot (frame == 40,inc(frame,1));
mul(ypos,frame,frameheight);
txtadd(crop,'0|',get(ypos),'|',get(framewidth),'|',get(frameheight));
delayedcall(0.06, animate() );
);
</action>


onout"set(frame,1);set(delaykilled,true)"

and before you animate();
set(delaykilled,false);animate();

Nothing really xml code here.. its just basic programming. My way is straightforward logic. I'm sure there is a cleverer method.
Try reading some programming tutorials online and try to stop trial and error in favor of very slow understanding of what you are trying to achieve and think it through step by step. That's the only way to learn. I rebelled against learning to programming anything until about 6 years ago.. It was a futile effort.

go back to your animate action and
onout="breakall();mul(ypos,1,frameheight);txtadd(crop,'0|',get(ypos),'|',get(framewidth),'|',get(frameheight));"

would be even better since it would stop the running action, and reset the frame at the same time.
Unless you have other actions running, then you have to go back to killing the action personally like I did before.

<action name="animate">
ifnot (frame == 40,inc(frame,1));
mul(ypos,frame,frameheight);
txtadd(crop,'0|',get(ypos),'|',get(framewidth),'|',get(frameheight));
delayedcall(0.06, animate() );
</action>