search

choderalab / openmoltools

An open set of tools for automating tasks relating to small molecules
MIT License
63 stars 30 forks source link

Fix #271 #272

Closed jchodera closed 5 years ago

jchodera commented 6 years ago

This fixes #271

jchodera commented 6 years ago

We should probably get this merged.

jchodera commented 6 years ago

@lnaden: Could you help with enabling osx tests here?

jchodera commented 5 years ago

@pgrinaway Looks like tests are re-running. Can you merge when done?

jchodera commented 5 years ago

I'm not sure we ever correctly enabled the osx tests. Those might have to be fixed separately. There were no osx tests previously, which is how this issue cropped up.

pgrinaway commented 5 years ago

Yep, I just wanted to see how the tests go before merging.

pgrinaway commented 5 years ago

I'm not sure we ever correctly enabled the osx tests. Those might have to be fixed separately.

Yeah, I think you're right.

zhang-ivy commented 5 years ago

@jchodera the openmoltools tests are failing on travis (see 1326.2 and 1326.3) and when I run them locally (after merging this commit with master)