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

Trim equilibration during analysis #90

Merged
merged 27 commits into from
May 2, 2023
Merged

Trim equilibration during analysis #90

merged 27 commits into from
May 2, 2023

Conversation

candidechamp
Copy link
Collaborator

@candidechamp candidechamp commented Apr 19, 2023

Description

The main objective behind these changes is to perform the "trimming" of the trajectories during the analysis step, rather than by submitting the parts which were removed for equilibration as a separate set of jobs.

The purpose behind those changes are twofold:

  • Remove input parameter for a lot of the submission functions to clarify what they are doing.
  • Simplify submission such that the calculations don't need to be broken down into too many sub-jobs.

NOTE: It is impossible to get perfect reproduction of results when comparing the old code (job submitted into 10 sub-jobs, including 1 for equilibration) and the new code (single larger job from which the first 10% is trimmed during analysis). This is because of numerical differences which make the simulation diverge. However average results and general behavior can be compared.

Todos

Notable points that this PR has either accomplished or will accomplish.

  • test code

  • Completely remove the input parameters separating "equilibration" and "production" runs?

This second bullet point will be done at a later stage if we find it necessary.

reeds/tests/REEDS_sopt/test_reeds_sopt_wrap.py Dismissed Show dismissed Hide dismissed
reeds/tests/REEDS_sopt/test_reeds_sopt_wrap.py Dismissed Show dismissed Hide dismissed
reeds/tests/REEDS_sopt/test_reeds_sopt_wrap.py Dismissed Show dismissed Hide dismissed
Candide Champion added 2 commits April 19, 2023 15:38
@codecov-commenter
Copy link

codecov-commenter commented Apr 19, 2023

Codecov Report

Merging #90 (b7154d9) into main (33ef630) will not change coverage.
The diff coverage is 28.94%.

@@           Coverage Diff           @@
##             main      #90   +/-   ##
=======================================
  Coverage   15.72%   15.72%           
=======================================
  Files          51       51           
  Lines        5534     5534           
=======================================
  Hits          870      870           
  Misses       4664     4664           
Flag Coverage Δ
unittests 15.72% <28.94%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...s/function_libs/analysis/parameter_optimization.py 0.00% <0.00%> (ø)
reeds/function_libs/analysis/replica_exchanges.py 0.00% <0.00%> (ø)
reeds/function_libs/analysis/sampling.py 45.39% <ø> (ø)
reeds/function_libs/pipeline/module_functions.py 0.00% <0.00%> (ø)
...cripts/analysis_workers/RE_EDS_general_analysis.py 0.00% <0.00%> (ø)
...ipts/analysis_workers/RE_EDS_optimization_final.py 0.00% <0.00%> (ø)
reeds/modules/do_RE_EDS_eoffEstimation.py 0.00% <ø> (ø)
reeds/modules/do_RE_EDS_eoffRebalancing.py 0.00% <ø> (ø)
reeds/modules/do_RE_EDS_generateOptimizedStates.py 0.00% <ø> (ø)
reeds/modules/do_RE_EDS_mixedOptimisation.py 0.00% <ø> (ø)
... and 4 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@candidechamp candidechamp merged commit c5bf047 into main May 2, 2023
5 checks passed
@candidechamp candidechamp deleted the trim_equil branch May 2, 2023 12:12
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

2 participants