Skip to content

Tweak example to be resumable - #21

Open
Delaunay wants to merge 6 commits into
masterfrom
resume_from_trials
Open

Tweak example to be resumable#21
Delaunay wants to merge 6 commits into
masterfrom
resume_from_trials

Conversation

@Delaunay

@Delaunay Delaunay commented Nov 9, 2022

Copy link
Copy Markdown
Collaborator

No description provided.

@Delaunay

Delaunay commented Nov 9, 2022

Copy link
Copy Markdown
Collaborator Author

maybe a less involved mechanism is required
more brainless checkpointing
I feel we might need like a checkpointer that would manage a single directory
and deals with that trial should use as checkpoint

Pierre Delaunay added 2 commits November 9, 2022 13:26
@codecov

codecov Bot commented Nov 9, 2022

Copy link
Copy Markdown

Codecov Report

Merging #21 (3ec1052) into master (be581e8) will increase coverage by 0.05%.
The diff coverage is 100.00%.

❗ Current head 3ec1052 differs from pull request most recent head c60e8d4. Consider uploading reports for the commit c60e8d4 to get more accurate results

@@            Coverage Diff             @@
##           master      #21      +/-   ##
==========================================
+ Coverage   97.02%   97.08%   +0.05%     
==========================================
  Files           4        4              
  Lines         404      412       +8     
  Branches       51       52       +1     
==========================================
+ Hits          392      400       +8     
  Misses          8        8              
  Partials        4        4              
Flag Coverage Δ
unittests 97.08% <100.00%> (+0.05%) ⬆️

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

Impacted Files Coverage Δ
hydra_plugins/hydra_orion_sweeper/config.py 100.00% <100.00%> (ø)
...ydra_plugins/hydra_orion_sweeper/implementation.py 96.41% <100.00%> (+0.07%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@Delaunay
Delaunay force-pushed the resume_from_trials branch from ff72259 to c60e8d4 Compare January 6, 2023 18:42
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.

1 participant