search

vlad17 / mve

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

sensible reporting names for dynamics metrics #318

Closed vlad17 closed 6 years ago

vlad17 commented 6 years ago

dynamics metrics have a bit of a weird naming setup -- every log(prefix) call logs the dynamics metrics under the name prefix + 'dynamics/open loop'.

Make this just prefix, which is way more expected, and also at all the DynamicsMetrics.log call sites change the prefix to something sensible (e.g., 'pre-iteration dynamics' and 'post-iteration dynamics' in DDPG, and something equally semantically valuable for the random shooter)

vlad17 commented 6 years ago

this is no longer relevant under new evaluation structure.