Adriano Meligrana
Adriano Meligrana
I added review comments about what we discussed @bergio13 @Datseris in the meeting as well as other example code simplifications
I have taken care of the most part of these review things myself. I've also generated these videos showing how random agents differ from RL agents in our example https://github.com/user-attachments/assets/fdf445a8-d833-46f8-a050-fa6bb48e7d8b...
The package has a totally different interface, so using that directly is a breaking change. In the case of Agents, it won't because we have a different interface on top...
The user is in total control of the union-splitting behaviour, before it was just on `getproperty/setproperty!`, while with this, through the `@unionsplit` macro, it allows to create branches automatically for...
yes, we can also improve the performance of `agent_step!` in ways not possible before. E.g. this would be better than before: `agent_step!(a::MultiAgent) = @unionsplit agent_step!(a)`
Yeah I think that can work, I would then need to specialize the operations (e.g. `delete!`) myself though right?