Skip to content

Conversation

@ijpulidos
Copy link
Collaborator

@ijpulidos ijpulidos commented Jul 19, 2023

Created a new PR with these changes such that we can use the CI secrets from our repo. Supersedes #271

These changes add template_generator_kwargs to the system generators.

Resolves #245 . Resolves #263 . Resolves #282

TODO

  • add tests

@ijpulidos ijpulidos added this to the 0.11.3 milestone Jul 19, 2023
@codecov-commenter
Copy link

codecov-commenter commented Sep 22, 2023

Codecov Report

Attention: 24 lines in your changes are missing coverage. Please review.

Comparison is base (3970498) 69.35% compared to head (b3b2886) 67.95%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #293      +/-   ##
==========================================
- Coverage   69.35%   67.95%   -1.40%     
==========================================
  Files           5        5              
  Lines         855      877      +22     
==========================================
+ Hits          593      596       +3     
- Misses        262      281      +19     
Files Coverage Δ
openmmforcefields/generators/system_generators.py 50.87% <100.00%> (+0.28%) ⬆️
...penmmforcefields/generators/template_generators.py 72.88% <17.24%> (-2.20%) ⬇️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jchodera
Copy link
Member

@mikemhenry @ijpulidos : I've finished cleaning up the docs and running local tests.
After this passes tests, can you merge and cut a new release?

@jchodera jchodera merged commit 6fe5f25 into main Oct 12, 2023
@jchodera jchodera deleted the fix/system_template_generator branch October 12, 2023 01:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

6 participants