Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Attempting a pull request to evaluate GH Actions workflow changes for more unittests #341

Merged
merged 12 commits into from
Jul 4, 2023

Commits on Jul 4, 2023

  1. attempt to address #340 by restricting max-workers to 1 (in attempt t…

    …o get more advanced unittests to work in GitHub actions, as parallelisation may be causing problems; have to commit to see if this will work!)
    carlhiggs committed Jul 4, 2023
    Configuration menu
    Copy the full SHA
    fbcbd70 View commit details
    Browse the repository at this point in the history
  2. Update tests.yml

    attempt to address #340 by restricting max-workers to 1 (attempting to edit online as local commit push failed with alert of limitations of personal access token for editing workflows)
    carlhiggs authored Jul 4, 2023
    Configuration menu
    Copy the full SHA
    4959872 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    e36f9bd View commit details
    Browse the repository at this point in the history
  4. Update tests.yml

    reverting because setting max-workers didn't resolve the issue #340
    carlhiggs authored Jul 4, 2023
    Configuration menu
    Copy the full SHA
    b7e31be View commit details
    Browse the repository at this point in the history
  5. implemented subprocesses as functions of ghsci to provide more flexib…

    …ility for test running (ie. without having to run these as subprocesses, in case this helps address #340)
    carlhiggs committed Jul 4, 2023
    Configuration menu
    Copy the full SHA
    b7dbfc0 View commit details
    Browse the repository at this point in the history
  6. further towards #340 (and removed a print statement from the neighbou…

    …rhood analysis code that was previously used in debugging)
    carlhiggs committed Jul 4, 2023
    Configuration menu
    Copy the full SHA
    fb21bce View commit details
    Browse the repository at this point in the history
  7. added an explicit test for creating a database (implicitly tests db c…

    …onnection); also added failfast=True to help move towards better #340
    carlhiggs committed Jul 4, 2023
    Configuration menu
    Copy the full SHA
    e12c62a View commit details
    Browse the repository at this point in the history
  8. Configuration menu
    Copy the full SHA
    b4fbe73 View commit details
    Browse the repository at this point in the history
  9. Configuration menu
    Copy the full SHA
    8f5784b View commit details
    Browse the repository at this point in the history
  10. replaced the full sensitivity analysis with a cheeky shortcut (a copy…

    … of the reference results are saved to the comparison path; suprise! they are identical!)
    carlhiggs committed Jul 4, 2023
    Configuration menu
    Copy the full SHA
    0220119 View commit details
    Browse the repository at this point in the history
  11. added back in writing of comparison config file for sensitivity analy…

    …sis test (inadvertently removed) towards #340
    carlhiggs committed Jul 4, 2023
    Configuration menu
    Copy the full SHA
    c92107a View commit details
    Browse the repository at this point in the history
  12. updated tests.py sensitivity analysis to mock up a more realistic com…

    …parison (cast the reference df to ints, so by rounding there is a difference); as per #340, the sensitivity analysis test is now fast and passes
    carlhiggs committed Jul 4, 2023
    Configuration menu
    Copy the full SHA
    bd51395 View commit details
    Browse the repository at this point in the history