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

Implement own version of Duvenauds Grammar #36

Open
3 tasks
BracketJohn opened this issue Jul 16, 2018 · 1 comment
Open
3 tasks

Implement own version of Duvenauds Grammar #36

BracketJohn opened this issue Jul 16, 2018 · 1 comment
Labels
enhancement New feature or request epic: kerndisc alpha Needs to be addressed before open sourcing. invalid This doesn't seem right

Comments

@BracketJohn
Copy link
Owner

There are some things that occurred during development and evaluation that might be improvements on the grammar of Duvenaud et al.
These improvements also don't seem to be covered in Lloyd et al.

It might be good to implement those in a modified version of duvenauds grammar, some small changes were already made that move away from Duvenauds design, these must then also be moved.

Tasks

  • Create new _grammar_duvenaud_modified.py grammar
  • Implement/Move over base kernel exclusion
    (rationale: Stops GPs from just taking constant or white as the "best" result)
  • Maybe make initial full expansion part of this grammar? Current implementation as part of discover might be unreasonable

TODO: Add more improvements noticed during development here.

@BracketJohn BracketJohn added enhancement New feature or request invalid This doesn't seem right epic: kerndisc alpha Needs to be addressed before open sourcing. labels Jul 16, 2018
@BracketJohn
Copy link
Owner Author

One of the biggest reasons to do this currently is the full initial expansion. It seems to be a grammar specific thing atm, but introduces a lot of noise trough out the code.

Another option would also be to just improve this behavior in general.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request epic: kerndisc alpha Needs to be addressed before open sourcing. invalid This doesn't seem right
Projects
None yet
Development

No branches or pull requests

1 participant