Open
Description
In the definition of Module::actions, the argument actions: &mut Vec<Self::Action> is just used to be pushed the result in. This makes no sense because it should be the caller to decide how to use the returned actions, and it also introduces a mut reference.
Instead, I'd like the definition of Module::actions to be
fn actions(&self, state: &Self::State) -> Vec<Self::Action>, and it could also make the API consistent with next_steps and next_states
Metadata
Metadata
Assignees
Labels
No labels