openvax / mhcflurry

Peptide-MHC I binding affinity prediction
http://openvax.github.io/mhcflurry/
Apache License 2.0
191 stars 57 forks source link

Fixing code typo: allelle -> allele. #108

Closed julia326 closed 6 years ago

timodonnell commented 6 years ago

LGTM thanks @julia326 . Does this option work now?

julia326 commented 6 years ago

Not yet, just filed an issue about that: #109

coveralls commented 6 years ago

Coverage Status

Coverage remained the same at 69.066% when pulling c4f42dd3a439abeb33a82a7127004b599a0685c4 on fix-allelle-typo into b3350b8ac81d445daf92108579e76d026a64e168 on master.

coveralls commented 6 years ago

Coverage Status

Coverage remained the same at 69.066% when pulling c4f42dd3a439abeb33a82a7127004b599a0685c4 on fix-allelle-typo into b3350b8ac81d445daf92108579e76d026a64e168 on master.

coveralls commented 6 years ago

Coverage Status

Coverage decreased (-0.2%) to 68.904% when pulling 7cd4b7952a508754ca0f8e1bc9b6732d07dd9b21 on fix-allelle-typo into b3350b8ac81d445daf92108579e76d026a64e168 on master.

coveralls commented 6 years ago

Coverage Status

Coverage decreased (-0.2%) to 68.904% when pulling 7cd4b7952a508754ca0f8e1bc9b6732d07dd9b21 on fix-allelle-typo into b3350b8ac81d445daf92108579e76d026a64e168 on master.

julia326 commented 6 years ago

@timodonnell the option works now - let me know if this is okay to merge, I added a bit to create the output models directory if it doesn't exist

timodonnell commented 6 years ago

Sure, LGTM!