update of github actions

jkarolczak / ligand-classification

Project examing sparse deep learning architectures for ligand classification.

3 stars 0 forks source link

update of github actions #111

Closed wtaisner closed 1 year ago

wtaisner commented 1 year ago

general idea is to test the following functionalities with github actions:

[ ] install ME for tests to pass
[ ] test, whether we can check building of docker containers/compse

jkarolczak commented 1 year ago

We cannot use Cuda on GitHub-hosted runners, because they run on machines without GPU. Thus we need our own runner to use Cuda. We cannot satisfy this requirement. Thus I recommend using @pytest.mark.skip as the universal issue resolver

wtaisner commented 1 year ago

Should we discard the issue & branch, or do we want to incorporate ME into our branch?

jkarolczak commented 1 year ago

afaik we don't test anything depending on ME, so imo we can get rid of installing ME. It should speed up tests.

wtaisner commented 1 year ago

agree, closing the issue as resolved