Starfish-develop / Starfish

Tools for Flexible Spectroscopic Inference
https://starfish.readthedocs.io/en/latest/
BSD 3-Clause "New" or "Revised" License
69 stars 22 forks source link

Tests cleanup #113

Closed mileslucas closed 5 years ago

mileslucas commented 5 years ago

This cleans up a lot of the unit tests, fixes regressions in the travis script as well as fixing read the docs auto-deployment. I managed to extremely bungle my git branches so this has one commit with a lot of changes from a manual reset.

Changelist:

tests

.travis.yml:

Note I ran into the same issues with timing out on the phoenix downloads. I opened a support ticket with travis ci to see if they had any ideas what was happening. To remedy this, in the meantime, I have marked the tests requiring this files to skip until it is resolved. I can confirm that the tests run locally as written. (#108)

read the docs

Some larger picture things-

I have decided to make the minimum required python version 3.6. I have two main reasons for this-

  1. I kept experiencing some really awkward regression failures on python 3.3-3.5, which required solutions that would greatly increase the complexity of simple operations like file saving
  2. Python 3.6 includes format strings, which would help clean up a lot of the code, if we can enforce python >= 3.6 we can take advantage of this across the code.

I don't think this kind of change should be that big of an issue. I've never ran into an issue with requiring 3.6 in my other projects and if people are installing from pip it will all be handled for us automatically anyways. I've written changes into the README.md and docs that show this change as well, for consistency.

iancze commented 5 years ago

This looks fantastic, thanks! Happy to add you as a contributor to the RTD, though I had trouble finding mileslucas there. Do you have a different username?

mileslucas commented 5 years ago

I had an old handle on a non-professional email, I just made a new account under mileslucas so give it a try again.