facebookresearch / phyre

PHYRE is a benchmark for physical reasoning.
https://phyre.ai
Apache License 2.0
431 stars 64 forks source link

Possible issue with simulate_single() #20

Closed pizilber closed 4 years ago

pizilber commented 4 years ago

In action_simulator.py, line 238, the function simulate_single() ends with return simulation.simulation_status, simulation.images and I think it's should be corrected to return simulation.status, simulation.images since Simulation has now a status attribute and not simulation_status.

Thanks.

lauragustafson commented 4 years ago

Fixed and version bumped to v0.2.1. Thanks for pointing this out!