Closed dotsdl closed 8 months ago
Pausing on this until #4 is in.
All modified lines are covered by tests :white_check_mark:
:exclamation: No coverage uploaded for pull request base (
main@cf0c2de
). Click here to learn what that means.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@ijpulidos any additional questions on this PR? I may want to start working on #2 soon, and we'd want to build on this one in building that out.
I think we're ready to merge this!
We want to separate out the creation of
system
,state
, andintegrator
from theSimulationUnit
in theNonEquilibriumCyclingProtocol
in order to enable the swapping in of alternateProtocolUnit
s for theSimulationUnit
, for example for use with Folding@Home viaalchemiscale-fah
. This PR aims to accomplish this.