Skip to content

Commit

Permalink
Removing everything related to fixing tests
Browse files Browse the repository at this point in the history
  • Loading branch information
Federico-PizarroBejarano committed May 14, 2024
1 parent cab87d3 commit 1e37320
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 23 deletions.
17 changes: 0 additions & 17 deletions .github/workflows/push.yml
Original file line number Diff line number Diff line change
Expand Up @@ -26,23 +26,6 @@ jobs:
pre-commit install
pre-commit autoupdate
pre-commit run --all
- name: Setup MySQL Server
env:
DB_USER: root
DB_PASSWORD: root
run: |
sudo apt-get install mysql-server
sudo systemctl start mysql
echo "REACHED HERE"
sudo mysql -h 127.0.0.1
CREATE USER optuna@"%";
CREATE DATABASE ppo_hpo;
CREATE DATABASE sac_hpo;
CREATE DATABASE gp_mpc_hpo;
GRANT ALL ON ppo_hpo.* TO optuna@"%";
GRANT ALL ON sac_hpo.* TO optuna@"%";
GRANT ALL ON gp_mpc_hpo.* TO optuna@"%";
exit
- name: Unit tests
run: |
python -m pytest tests/
Expand Down
2 changes: 0 additions & 2 deletions tests/test_hpo/test_hpo.py
Original file line number Diff line number Diff line change
Expand Up @@ -176,8 +176,6 @@ def test_hpo_without_database(SYS, TASK, ALGO, SAMPLER):
'''Test HPO for one single trial without using MySQL database.
(create a study from scratch)
'''
pytest.skip('Takes too long.')

# output_dir
output_dir = './examples/hpo/results'
# delete output_dir
Expand Down
5 changes: 1 addition & 4 deletions tests/test_hpo/test_train.py
Original file line number Diff line number Diff line change
Expand Up @@ -14,10 +14,7 @@
@pytest.mark.parametrize('ALGO', ['ppo', 'sac', 'gp_mpc'])
@pytest.mark.parametrize('HYPERPARAMETER', ['default', 'optimimum'])
def test_train(SYS, TASK, ALGO, HYPERPARAMETER):
'''Test training rl/lbc given a set of hyperparameters.
'''
pytest.skip('Takes too long.')

'''Test training rl/lbc given a set of hyperparameters.'''
# output_dir
output_dir = './examples/hpo/results'
# delete output_dir if exists
Expand Down

0 comments on commit 1e37320

Please sign in to comment.