Re: [simspark-devel] About SoccerRules

[Joschka B. <jbo...@un...>]

Hi Hedayat,

Hedayat Vatankhah wrote:
>
> I've changed MoveAgent for the new agent (thanks BeamEffector!) and 
> fixed ClearPlayers functions.
>

Great, thank you. I've just committed a bunch of changes to the 
SoccerBase (mainly to increase performance) and I've moved the MoveAgent 
code to this class. This way, it is accessible to other classes, too. 
I'm using it in the BeamEffector, the SoccerRuleAspect, and the 
TrainerCommandParser now.

By the way, the TrainerCommandParser should be ready to be used with 
simspark now, so hopefully, this should make testing the rule stuff a 
little easier :-)

> I don't know if I'm going in the right direction or not! It seems that 
> most of the old code can be used with little or no modification 
> (UpdateKickOff for example). Am I right?!!
>
>

Yes, you're right. Most of the code should work now, so we can 
re-activate the commented out portions of the code little by little and 
test (I already activated the UpdateKickOff part in the SIMSPARK_RULES 
branch of the CVS). One of my main concerns was the performance drop the 
integration of the rules might bring, but I hope my latest commits might 
help to deal with that (crossing my fingers that I didn't break anything 
;-) ).

I'm now working on a new SimControlNode that allows to display GameState 
info in the internal monitor since I'm having a hard time with monitorspark.

If you implement the BoundingBox methods for the agent, maybe you can 
put them in the SoccerBase class. I think they might be useful for other 
classes, too.

Cheers,
Joschka