Alembic migrations tests on github actions work by checking main, installing dependencies then running tests.
The issue with that is when a PR is raised with a new dependency, the test will fail.
This PR also explicitly adds sklearn as a dependency.
Goal
Changes
Future Tasks (optional)
How has this been tested?
To-do before merge (optional)
Checklist
Fill with x for completed.
[ ] My code follows the style guidelines of this project
[ ] I have reviewed my own code to ensure good quality
[ ] I have tested the functionality of my code to ensure it works as intended
[ ] I have resolved merge conflicts
(Delete any items below that are not relevant)
[ ] I have updated the automated tests
[ ] I have updated the scripts in scripts/
[ ] I have updated the requirements
[ ] I have updated the README file
[ ] I have updated affected documentation
[ ] I have added a blogpost in Latest Updates
[ ] I have updated the CI/CD scripts in .github/workflows/
Reviewer: @tonyzhao6 Estimate: 15mins
Ticket
Fixes:HOTFIX
Description
Alembic migrations tests on github actions work by checking main, installing dependencies then running tests. The issue with that is when a PR is raised with a new dependency, the test will fail. This PR also explicitly adds sklearn as a dependency.
Goal
Changes
Future Tasks (optional)
How has this been tested?
To-do before merge (optional)
Checklist
Fill with
xfor completed.(Delete any items below that are not relevant)
scripts/.github/workflows/