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

Component lock #120

Merged
merged 2 commits into from
Apr 10, 2017
Merged

Component lock #120

merged 2 commits into from
Apr 10, 2017

Conversation

kengz
Copy link
Owner

@kengz kengz commented Apr 10, 2017

We have a lot of components, and not all of them are compatible with another. When scheduling experiments and designing specs it is hard to keep all of them in check.

This adds a component locks that does automatic checking of all specs when importing, by using the specified locks in rl/spec/component_locks.json. Uses the minimum description length design principle. When adding new components, be sure to update this file.

  • add double-network component lock
  • add discrete-action component lock; assume continuous agent can handle discrete action spaces as a generalization

@kengz kengz merged commit aadb4c4 into master Apr 10, 2017
@kengz kengz deleted the component-lock branch April 10, 2017 01:31
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.

1 participant