keiohta / tf2rl

TensorFlow2 Reinforcement Learning
MIT License
461 stars 104 forks source link

Unused argument `max_steps' in the function restore_latest_n_traj #144

Closed naji-s closed 2 years ago

naji-s commented 2 years ago

The body of the function restore_latest_n_traj does not use the argument max_steps as can be seen here: https://github.com/keiohta/tf2rl/blob/82d9eecda78e22021efa0821bf02429ac7827f4d/tf2rl/experiments/utils.py#L13

What I do think needs to happen is to change https://github.com/keiohta/tf2rl/blob/82d9eecda78e22021efa0821bf02429ac7827f4d/tf2rl/experiments/utils.py#L16

to return load_trajectories(filenames, max_steps) or am I missing something?