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.

Sign up for GitHub

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

Jump to bottom

Save and Load MABs #185

Merged
merged 31 commits into from
Sep 12, 2023
Merged

Save and Load MABs #185

merged 31 commits into from
Sep 12, 2023

Conversation

maypink
Copy link
Collaborator

@maypink maypink commented Sep 7, 2023 •
edited
Loading

Add save and load methods for MAB.

maypink added 20 commits March 17, 2023 11:52
@maypink
fix
ff02744
@maypink
fix#2
987cce0
@maypink
minor
022363c
@maypink
Merge branch 'main' of https://github.com/aimclub/GOLEM into 66-singl…
d24dfa2 
…e_drop_mutation-fails-to-remove-node
@maypink
Merge branch 'main' of https://github.com/aimclub/GOLEM
4900e5f
@maypink
Merge branch 'main' of https://github.com/aimclub/GOLEM
cc8729f
@maypink
Merge branch 'main' of https://github.com/aimclub/GOLEM
f320cfa
@maypink
Merge branch 'main' of https://github.com/aimclub/GOLEM
f3ca604
@maypink
Merge branch 'main' of https://github.com/aimclub/GOLEM
b76b1c3
@maypink
Merge branch 'main' of https://github.com/aimclub/GOLEM
13a76bf
@maypink
Merge branch 'main' of https://github.com/aimclub/GOLEM
df115e3
@maypink
Merge branch 'main' of https://github.com/aimclub/GOLEM
4342573
@maypink
Merge branch 'main' of https://github.com/aimclub/GOLEM
56db3a7
@maypink
Merge branch 'main' of https://github.com/aimclub/GOLEM
5826890
@maypink
Merge branch 'main' of https://github.com/aimclub/GOLEM
33339ef
@maypink
Merge branch 'main' of https://github.com/aimclub/GOLEM
1863a1a
@maypink
Merge branch 'main' of https://github.com/aimclub/GOLEM
da4eaad
@maypink
Merge branch 'main' of https://github.com/aimclub/GOLEM
0959a02
@maypink
Merge branch 'main' of https://github.com/aimclub/GOLEM
9543642
@maypink
Merge branch 'main' of https://github.com/aimclub/GOLEM
a9885f4
@maypink maypink linked an issue Sep 7, 2023 that may be closed by this pull request
Add saving option for MAB #170
Closed
@aim-pep8-bot
Copy link
Collaborator

aim-pep8-bot commented Sep 7, 2023 •
edited
Loading

Hello @maypink! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2023-09-12 09:53:08 UTC

@codecov-commenter
Copy link

codecov-commenter commented Sep 7, 2023 •
edited
Loading

Codecov Report

Merging #185 (cb27961) into main (147df93) will increase coverage by 0.19%.
Report is 5 commits behind head on main.
The diff coverage is 93.33%.

@@            Coverage Diff             @@
##             main     #185      +/-   ##
==========================================
+ Coverage   74.10%   74.30%   +0.19%     
==========================================
  Files         129      129              
  Lines        7678     7717      +39     
==========================================
+ Hits         5690     5734      +44     
+ Misses       1988     1983       -5
Files Changed Coverage Δ
...olem/core/optimisers/genetic/operators/mutation.py 89.21% <ø> (-1.79%) ⬇️
...m/core/optimisers/adaptive/mab_agents/mab_agent.py 77.27% <93.10%> (+36.73%) ⬆️
golem/core/optimisers/optimization_parameters.py 97.14% <100.00%> (+0.08%) ⬆️

... and 5 files with indirect coverage changes

@maypink maypink marked this pull request as ready for review September 8, 2023 13:40
@maypink maypink requested a review from YamLyubov September 8, 2023 13:42
@maypink maypink requested a review from YamLyubov September 11, 2023 14:22
@maypink maypink force-pushed the 170-add-saving-option-for-mab branch from f75502b to b18f159 Compare September 12, 2023 09:51
@maypink maypink merged commit a5c4e56 into main Sep 12, 2023
@maypink maypink deleted the 170-add-saving-option-for-mab branch September 12, 2023 09:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@YamLyubov YamLyubov YamLyubov approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add saving option for MAB
4 participants
@maypink @aim-pep8-bot @codecov-commenter @YamLyubov