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

Fbopt_depreciated #353

Closed
wants to merge 254 commits into from
Closed

Fbopt_depreciated #353

wants to merge 254 commits into from

Conversation

smilesun
Copy link
Collaborator

@smilesun smilesun commented Sep 8, 2023

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Sep 8, 2023

Codecov Report

Merging #353 (2276ec9) into master (f84c77d) will decrease coverage by 2.01%.
The diff coverage is 65.49%.

@@            Coverage Diff             @@
##           master     #353      +/-   ##
==========================================
- Coverage   95.35%   93.35%   -2.01%     
==========================================
  Files         131      136       +5     
  Lines        4633     4968     +335     
==========================================
+ Hits         4418     4638     +220     
- Misses        215      330     +115     
Flag Coverage Δ
unittests 93.35% <65.49%> (-2.01%) ⬇️

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

Files Changed Coverage Δ
domainlab/algos/trainers/train_hyper_scheduler.py 100.00% <ø> (ø)
domainlab/algos/trainers/fbopt.py 16.48% <16.48%> (ø)
domainlab/algos/msels/c_msel_bang.py 62.50% <62.50%> (ø)
domainlab/compos/exp/exp_utils.py 98.33% <66.66%> (-0.82%) ⬇️
domainlab/algos/trainers/train_fbopt.py 79.59% <79.59%> (ø)
domainlab/algos/trainers/fbopt_alternate.py 81.91% <81.91%> (ø)
domainlab/algos/observers/c_obvisitor_cleanup.py 89.47% <85.71%> (-2.84%) ⬇️
domainlab/algos/trainers/a_trainer.py 96.00% <100.00%> (+0.28%) ⬆️
domainlab/algos/trainers/args_fbopt.py 100.00% <100.00%> (ø)
domainlab/algos/trainers/zoo_trainer.py 91.66% <100.00%> (+0.75%) ⬆️
... and 8 more

... and 1 file with indirect coverage changes

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

domainlab/algos/trainers/fbopt.py Outdated Show resolved Hide resolved
mmu = None
for miter in range(self.budget_mu_per_step):
# FIXME: the same mu is tried two times since miter=0
mmu = self.dict_addition(self.mmu, miter * self.delta_mu)
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

miter always start from 0, but let miter=0 only make sense for the 1st iteration

@smilesun smilesun changed the title Fbopt Fbopt_depreciated Sep 23, 2023
@smilesun
Copy link
Collaborator Author

fb_master is not need anymore, this pr will be closed

@smilesun smilesun closed this Sep 23, 2023
@smilesun smilesun deleted the fbopt_master branch October 7, 2023 13:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fbopt priority Further information is requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants