nsat / pypredict

Spire port of predict open-source tracking library
GNU General Public License v2.0
64 stars 17 forks source link

Add preliminary Travis-CI support #29

Closed Echelon9 closed 2 years ago

Echelon9 commented 6 years ago

This PR adds preliminary support for continuous integration, specifically via the Travis-CI platform.

Travis-CI is a cloud-hosted continuous integration platform, with good quality GitHub integration and free for open source projects.

How to enable:

Comments:

Neither of these two comments need to be fixed before this support lands IMHO, as they can be addressed later. In fact, having this skeleton in place provides a good base to build upon and provides confidence in future refactoring efforts.

If the testsuite is known failing at the moment, how do I know a working testsuite will run green? Good question. I quickly hacked around the above two problems which returned green on Python 2.6 and Python 2.7.

If anyone is still unconvinced of the benefits of CI, here is a short set of reasons why.

Echelon9 commented 4 years ago

I've rebased this PR on top of the master branch, and confirmed that the test suite runs still on Travis-CI infrastructure.

There remains a known test failure, whether run locally or on CI, due to above noted TLE staleness issue:

$ ./test.py

test_transits_are_truncated_if_the_overlap_the_start_or_end_times (__main__.TestPredict) ... ERROR

======================================================================

ERROR: test_transits_are_truncated_if_the_overlap_the_start_or_end_times (__main__.TestPredict)

----------------------------------------------------------------------

Traceback (most recent call last):

  File "./test.py", line 32, in test_transits_are_truncated_if_the_overlap_the_start_or_end_times

    next_transit = next(predict.transits(tle, qth, ending_after=at))

  File "/home/travis/build/Echelon9/pypredict/predict.py", line 66, in transits

    transit = quick_predict(tle, ts, qth)

  File "/home/travis/build/Echelon9/pypredict/predict.py", line 19, in quick_predict

    return _quick_predict(tle, ts, qth)

PredictException: time Wed 14Jan15 02:27:36 too far from present

----------------------------------------------------------------------

Ran 1 test in 0.002s
miketwo commented 2 years ago

I've addressed your second task with my PR. Locally, I used your test setup to develop the solution, so thank you for that.

nick-pascucci-spire commented 2 years ago

Hi, thanks for your pull request! We have recently added new maintainers to this project, including myself, and are addressing the outstanding PRs.

I've integrated GitHub Actions CI in #53 which is substantially similar to what you've proposed here, so I'm going to close this PR out.

Echelon9 commented 2 years ago

No worries. Good to see that the project is under continued maintenance. All the best.