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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

speed up and improve CI and update readme #169

Merged
merged 3 commits into from
Jul 13, 2023
Merged

Conversation

mikemhenry
Copy link
Contributor

No description provided.

@mikemhenry mikemhenry changed the title speed up and improve CI speed up and improve CI and update readme Jul 13, 2023
@codecov-commenter
Copy link

codecov-commenter commented Jul 13, 2023

Codecov Report

Merging #169 (9e3cbb4) into main (2e92ecb) will increase coverage by 56.63%.
The diff coverage is n/a.

@mikemhenry
Copy link
Contributor Author

mikemhenry commented Jul 13, 2023

env build time:

  • this pr: 3m
  • old: 3m 40s

test time:

  • this pr: 9m
  • old: 13m 20s

@mikemhenry mikemhenry added this to the 0.3.1 milestone Jul 13, 2023
@mikemhenry
Copy link
Contributor Author

Oh and now coverage reports are working: https://app.codecov.io/gh/choderalab/espaloma/pull/169/indirect-changes

@mikemhenry mikemhenry merged commit 116ced2 into main Jul 13, 2023
5 checks passed
@mikemhenry mikemhenry deleted the feat/speed_up_CI branch July 13, 2023 20:29
@mikemhenry
Copy link
Contributor Author

There is some buggy behavior in our tests (well not really but they are not deterministic so when we try and distribute the tests to different workers, we can get an error. I will fix this in a different PR but for now will turn off -n auto) in PR #172

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants