search

ersilia-os / ersilia

The Ersilia Model Hub, a repository of AI/ML models for infectious and neglected disease research.
https://ersilia.io
GNU General Public License v3.0
219 stars 147 forks source link

🐈 Task: Clean up GitHub Actions Workflows #1231

Open DhanshreeA opened 2 months ago

DhanshreeA commented 2 months ago

Summary

We have a lot of workflows in the repository currently, a lot of which are not even getting used, and some workflows are duplicated. For example,

  1. Install Ersilia and Run Pytest workflows could go into the same file as parallel jobs since they both have the same triggers. Test Colab could also go in the same file as a parallel job, or sequential to Run Pytest.
  2. Quick Model Test (manual), Ersilia model ouptut prediction (Manual), and Ersilia model output prediction - all seem to be doing the same thing, and they all require a manual trigger, even though the last one was supposed to run on a schedule at some point.
  3. Ersilia Base Image to DockerHub Upload and Ersilia Installer to DockerHub upload can both go into the same file as parallel jobs since they both have the same workflow trigger, and are independent of each other. There's a slight caveat, the trigger for these jobs depends on a workflow that no longer exists. Tagging and versioning now happen in the Ersilia Release workflow. On second thoughts, I think the DockerHub uploads can be bundled in this workflow as sequential jobs after the GH and PyPI releases are successful.

Objective(s)

TODO

Documentation

No response

dzumii commented 2 months ago

@DhanshreeA I have started working on this, do you advise i take each on different PRs? So i can easily test and not mess things up too much at a go.

GemmaTuron commented 1 month ago

Hi @DhanshreeA

There is a bunch of PR's linked to this. Can you update of the mentioned issues, which ones are remaining to tackle?

DhanshreeA commented 1 month ago

Hi @GemmaTuron we have managed to achieve point 1 in the summary provided above.