entropicalabs / openqaoa

Multi-backend SDK for quantum optimisation
MIT License
113 stars 59 forks source link

Split test_problems.py into test_*.py for each class #254

Closed Newtech66 closed 1 year ago

Newtech66 commented 1 year ago

Description

Checklist

Type of change

Please delete options that are not relevant.

How Has This Been Tested?

PortfolioOptimization has been added to test_QUBO.py, but it doesn't pass on my system because the change to helper_functions.py isn't reflecting on the install for some reason.

vishal-ph commented 1 year ago

@newton66, can you please update your branch by pulling in the latest changes from dev. We realized there was an issue with the test_ext_dev action script that prevented the tests inside src/*/tests/ folders to run. We have fixed this in the latest commit on dev. Once you have pulled these changes, we can run the tests again

Newtech66 commented 1 year ago

@vishal-ph I did it

vishal-ph commented 1 year ago

@Newtech66, I cannot see the new changes on your branch yet, could you please merge the branch once more? Thanks

vishal-ph commented 1 year ago

The only test that failed in this PR was due to IBM account credentials being inaccessible by forked PRs. The PR is good to merge