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

Add molecule encodings for contextual bandit #179

Merged
merged 14 commits into from
Sep 5, 2023
Merged

Conversation

YamLyubov
Copy link
Collaborator

@YamLyubov YamLyubov commented Aug 28, 2023

Adds molecular encoders for contextual graphs.
Implemented approaches:

Collected materials: https://docs.google.com/spreadsheets/d/1qDx6hhRMeA185_fLxBRwwcHHmvUOxbG9ID_IkOYZV-g/edit?usp=sharing

Results for 10 trials:
qed score
Figure_1

normalized sa score
Figure_2
Figure_3

#176

@aim-pep8-bot
Copy link
Collaborator

aim-pep8-bot commented Aug 28, 2023

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

Line 18:1: E402 module level import not at top of file

Comment last updated at 2023-09-05 12:37:59 UTC

@codecov-commenter
Copy link

codecov-commenter commented Aug 29, 2023

Codecov Report

Merging #179 (909d251) into main (d5d1dfa) will increase coverage by 0.10%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #179      +/-   ##
==========================================
+ Coverage   74.02%   74.12%   +0.10%     
==========================================
  Files         129      129              
  Lines        7688     7688              
==========================================
+ Hits         5691     5699       +8     
+ Misses       1997     1989       -8     
Files Changed Coverage Δ
golem/core/optimisers/genetic/gp_params.py 100.00% <100.00%> (ø)

... and 4 files with indirect coverage changes

@YamLyubov YamLyubov linked an issue Aug 30, 2023 that may be closed by this pull request
@YamLyubov YamLyubov requested a review from maypink September 4, 2023 08:47
examples/molecule_search/mol_encoders.py Outdated Show resolved Hide resolved
examples/molecule_search/mol_encoders.py Outdated Show resolved Hide resolved
examples/molecule_search/mol_encoders.py Show resolved Hide resolved
examples/molecule_search/mol_transformer/transformer.py Outdated Show resolved Hide resolved
examples/molecule_search/mol_transformer/transformer.py Outdated Show resolved Hide resolved
test/unit/adaptive/test_context_agents.py Outdated Show resolved Hide resolved
@YamLyubov YamLyubov requested a review from maypink September 5, 2023 12:46
@YamLyubov YamLyubov merged commit cd35325 into main Sep 5, 2023
@YamLyubov YamLyubov deleted the molecule-encoding branch September 5, 2023 15:49
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.

Implement graph encoder for contextual bandits on Molecule search
4 participants