Skip to content

Update CI for OPS 2.0 - #1043

Merged
dwhswenson merged 5 commits into
openpathsampling:dev-2.0from
dwhswenson:update-2.0-ci
Aug 5, 2021
Merged

Update CI for OPS 2.0#1043
dwhswenson merged 5 commits into
openpathsampling:dev-2.0from
dwhswenson:update-2.0-ci

Conversation

@dwhswenson

Copy link
Copy Markdown
Member

The main development branch CI is in .github/workflows/tests.yml. The CI for the 2.0 development branch is in .github/workflows/test-2.0.yml. This PR ports improvements in tests.yml into test-2.0.yml (and, of course, targets the dev-2.0 branch).

Useful improvements:

  • Adds Python 3.8 and 3.9 to the test suite. This wasn't possible (no OpenMM available) before.
  • Adds notebook tests. I'm not sure why this wasn't done before, but with core changes on the way, such as New iter_generate for OPS 2.0 #1042, I'll feel a lot better having the more thorough test suite.

CI for 2.0 was set up before OpenMM was available on > 3.7.
@dwhswenson dwhswenson added 2.0 issues and PRs for the 2.0 release misc PR labels Aug 2, 2021
I think the files used are old enough that they require workarounds
that were removed when deprecations were removed. If this fixes,
then I'll create new files for the dev-2.0 branch.
@dwhswenson

Copy link
Copy Markdown
Member Author

This also required creating new files for Python 3.7 that don't use old workarounds. The old files (created in Python 3.6 requiring workarounds due to changes in the structure of interface sets) are still used on the main branch.

This is ready for review and comment. I will leave it open for at least 24 hours, merging no earlier than Wed 04 Aug 00:00 GMT (Tue 03 Aug 20:00 my local).

@sroet sroet left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sroet sroet mentioned this pull request Aug 3, 2021
@dwhswenson
dwhswenson merged commit 565353f into openpathsampling:dev-2.0 Aug 5, 2021
@dwhswenson
dwhswenson deleted the update-2.0-ci branch August 5, 2021 17:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2.0 issues and PRs for the 2.0 release misc PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants