Skip to content

Implements LaMBO2 in poli-baselines #49

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

Merged
merged 20 commits into from
Sep 25, 2024
Merged

Implements LaMBO2 in poli-baselines #49

merged 20 commits into from
Sep 25, 2024

Conversation

miguelgondu
Copy link
Collaborator

@miguelgondu miguelgondu commented Sep 18, 2024

  • Bring the scaffold.
  • Implement an example.
  • Clean up the hydra configs.
  • Use overwrites instead of the config_path and config_name.
    • Maybe write a class with the relevant parameters and still keep overwrites.
    • Change initialization for an absolute path using initialize_config_dir and add config_path and config_name again.
  • Implement an example with updating the configuration.
  • Decide what to do when no data is provided.
    • An alternative: at least a value should be provided for x0. We can bootstrap from that using evostrats/directed evolution.
  • Add tests.
    • Wait for the wandb issue to be addressed in cortex.
    • In ehrlich, LaMBO2 sometimes proposes - instead of a residue... what should we do then?
  • Add documentation.

@miguelgondu miguelgondu marked this pull request as ready for review September 23, 2024 21:26
@RMichae1
Copy link
Collaborator

Can replicate example run locally.

@miguelgondu miguelgondu merged commit d6c1c2d into main Sep 25, 2024
7 checks passed
@miguelgondu miguelgondu deleted the feature-add-lambo2 branch September 25, 2024 11:30
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.

2 participants