utiasDSL / safe-control-gym

PyBullet CartPole and Quadrotor environments—with CasADi symbolic a priori dynamics—for learning-based control and RL
https://www.dynsyslab.org/safe-robot-learning/
MIT License
560 stars 123 forks source link

Fixing HPO Linting Pipeline Failure #152

Closed Federico-PizarroBejarano closed 2 months ago

Federico-PizarroBejarano commented 2 months ago

I didn't realize the HPO had three pipeline failures:

  1. Linting failed due to some minor issues
  2. test_hpo_database.py, test_hpo.py, and test_train.py require the SQL database to be set up, which needs to be added to the GitHub Actions.
  3. test_hpo.py and test_train.py take way too long, which needs to be shortened or excluded from GitHub Actions.

This PR specifically fixes linting issues, and the test issues will be left up to @middleyuan

middleyuan commented 2 months ago

Looks good to me.