search

vlad17 / mve

MVE: model-based value estimation
Apache License 2.0
10 stars 0 forks source link

Sampler fixes #309

Closed vlad17 closed 6 years ago

alvinwan commented 6 years ago

Just wanted to clarify: I'm currently running all 14 experiments off of this branch, the added flag todo commit https://github.com/vlad17/cmpc/pull/309/commits/b68feed418a7a5941822aa4376149af7424a371e. Is that all right? It looks like the max_horizon fix hasn't creeped into master yet.

vlad17 commented 6 years ago

Yes that was just a comment. Go ahead and run the commit you mentioned. On Fri, Feb 2, 2018 at 8:21 PM Alvin Wan notifications@github.com wrote:

Just wanted to clarify: I'm currently running all 14 experiments off of this branch, the added flag todo commit b68feed https://github.com/vlad17/cmpc/commit/b68feed418a7a5941822aa4376149af7424a371e. Is that all right? It looks like the max_horizon fix hasn't creeped into master yet.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/vlad17/cmpc/pull/309#issuecomment-362778459, or mute the thread https://github.com/notifications/unsubscribe-auth/AFkI1E2n70SlAlN0Qxf4wfmZlPp2S8x1ks5tQ968gaJpZM4R4BsH .