-
Notifications
You must be signed in to change notification settings - Fork 63
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
Added new reward_learning experiment #478
base: master
Are you sure you want to change the base?
Conversation
Great! You will need to fix the linting errors that are shown here https://circleci.com/gh/expfactory/expfactory-experiments/950 and then push to the branch to run the check again. When all tests pass, we can merge into the branch. |
Looks like listing is good - you need to add your new experiment to the circle.yml file. Just follow the others for example! |
Hey so for commit messages you might want to be specific about what the commit changed, so coming back in the future you can find changes more efficiently! |
Definitely agree! In this case, I'm just updating the list.js files On Thu, Jul 28, 2016 at 10:21 AM, Vanessa Sochat [email protected]
|
Hi!
I finally added my long-term reward learning experiment.