Closed decodyng closed 5 years ago
Re including test_traj.json: I think the best thing to do would be to include a set of test_trajectories that represent non-arbitrary things, but that might need to wait until you guys wrap up NeurIPS responses for Micah to have the time. In the absence of that, I lean towards keeping it in just so it works at all out of the box.
Re incompatible models and layouts: I suspect that there would be some error message on the backend/console, but agree that it wouldn't show up in the front end as currently coded. I'll add it as an issue!
LGTM, feel free to merge. (The conflict looks easy to resolve.)