search

mgalardini / pyseer

SEER, reimplemented in python 🐍🔮
http://pyseer.readthedocs.io
Apache License 2.0
104 stars 25 forks source link

Add test that phenotype is numeric #140

Closed johnlees closed 3 years ago

johnlees commented 3 years ago

Raised in #139

johnlees commented 3 years ago

@mgalardini I am still having trouble seeing the CI tests for PRs. Do you have any thoughts on maybe moving to github actions? I did this recently in PopPUNK (e.g. https://github.com/johnlees/PopPUNK/blob/master/.github/workflows/azure_ci.yml)

mgalardini commented 3 years ago

Hi John, sure we could do that! In the meantime if you click on the Travis CI badge you can see the tests for the branch you made, there seems to be a TypeError right now: TypeError: Cannot construct a dtype from an array

johnlees commented 3 years ago

Ok, I'll try adding that in to this PR too

mgalardini commented 3 years ago

I think it should be cd tests && bash run_test.sh? Thanks for setting this up, we can remove Travis in the next release then