allenai / allenact

An open source framework for research in Embodied-AI from AI2.
https://www.allenact.org
Other
318 stars 52 forks source link

Error with robothor_environment/cache_utils: retry_dist() takes 2 positional arguments but 3 were given #239

Closed ekolve closed 3 years ago

ekolve commented 3 years ago

This line:

https://github.com/allenai/allenact/blob/master/utils/cache_utils.py#L162

calls this function:

https://github.com/allenai/allenact/blob/master/plugins/robothor_plugin/robothor_environment.py#L139

which only expects 2 arguments, but 3 are given. This happens when training a pointnav model.

jordis-ai2 commented 3 years ago

I'm a bit surprised that a pointnav model is calling a function to compute distance to an object. You might have uncovered a bigger bug!

jordis-ai2 commented 3 years ago

Can you share some more details about the experiment you're running? After inspecting, I haven't seen any pointnavtask-related code calling the wrapping funtion of https://github.com/allenai/allenact/blob/master/plugins/robothor_plugin/robothor_environment.py#L139

jordis-ai2 commented 3 years ago

I'm tentatively fixing the call to native_distance_function (which should in any case expect two parameters), but I haven't been able to determine where in the code we use distance_to_object in a pointnav task.