utiasDSL / safe-control-gym

PyBullet CartPole and Quadrotor environments—with CasADi symbolic a priori dynamics—for learning-based control and RL
https://www.dynsyslab.org/safe-robot-learning/
MIT License
560 stars 123 forks source link

Moving compute_metrics to MetricExtractor #126

Closed Federico-PizarroBejarano closed 1 year ago

Federico-PizarroBejarano commented 1 year ago

Very minor change that I think is useful. I found that the metric extractor class is useful, but right now its only really accessible through the experiment. However, for my experiments I save the trajectory data and want to be able to compute the metrics on the fly. This minor change should make that easier, without changing the interface for the experiments. We could also make the MetricExtractor class completely static, but I tried and its pretty ugly. Not a necessary change but I think its useful.