afids / afids-validator

Validator for the anatomical fiducial placement protocol
https://validator.afids.io
GNU General Public License v3.0
2 stars 4 forks source link

BUG: Fix actions cache #139

Closed kaitj closed 3 years ago

kaitj commented 3 years ago

Proposed changes

The Github actions cache was previously failing to actually cache the environment resulting in downloading the libraries listed in requirements.txt for each run. This PR fixes this issue and also removes the virtual environment (included screenshot for successful run).

image

Alternatively, we can choose to still use a virtual environment have it install the necessary libraries from cache rather than cache the environment. For reference, it would require removing the "if" statement in the installation step. From there, it will check if a cache is found, and if it is found, install the library from cache.

Types of changes

What types of changes does your code introduce? Put an x in the boxes that apply

Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you are unsure about any of the choices, don't hesitate to ask!

Notes

All PRs will undergo the unit testing before being reviewed. You may be requested to explain or make additional changes before the PR is accepted.

_PR template was adopted from appium_