game-devs-csf / RL-environments

Environments for RL
MIT License
2 stars 1 forks source link

Model handling through `baseMdl` module #21

Closed BassedWarrior closed 9 months ago

BassedWarrior commented 9 months ago

Fixes #3, Fixes #4

BassedWarrior commented 9 months ago

Creo que hay varias cosas que si se aprueba este PR, se pueden mejorar, pero que dependend de que este PR se apruebe. Y aunque puede abordar #3 y #4, creo que no los cierra por completo. Ya que se debe hacer más trabajo para que en específico la interfaz de importar y exportar modelos sea mejor.

Un mejor manejo de los paths en los que se guarda y se carga los modelos, por ejemplo. Y un mejor manejo de los archivos. Ya que estos commits preservan archivos que se volverían fráncamente redundantes, como examples/cartpole_tests o examples/cartpole_import_export_models.

Sin embargo no se si se deben de hacer PRs DESPUÉS de que este se apruebe...o si se puede aumentar el scope de este...o se hacen PRs aparte...

A01352283 commented 9 months ago

Por fin me senté a checar tu PR, una disculpa por la tardanza jeje.