allenai / allenai/ai2thor

Merge Teleport and TeleportFull?

未关闭
#447 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
Agent Navigation Feature Request
主要语言
C#
星标
1.8k
派生
297
PR 合并指标
30 天内没有已合并 PR

描述

`TeleportFull` seems like a misuse of leveraging default key-word args.

---

For reference, `Teleport` is defined as
```python
controller.step(
action='Teleport',
x=0.999, y=1.01, z=-0.3541)
```
and `TeleportFull` is defined the same way, but with 2 extra args:
```python
event = controller.step(
action='TeleportFull',
x=0.999, y=1.01, z=-0.3541,
rotation=dict(x=0.0, y=90.0, z=0.0),
horizon=30.0)
```

---

What would seem easier from a user's perspective would be to allow users to **optionally** specify the `rotation`, `horizon`, and also `x`, `y`, `z` positions. And for all the options that are not specified, leave them **as is**.

For example, running
```python
controller.step(
action='Teleport',
x=0.999, z=-0.3541,
horizon=-30)
```
Will change only the x/z position and horizon of the agent, and leave the y/rotation as is. This will make it so that users don't have to know some crazy float `y` value that's only accessible from `GetReachablePositions`.

Effectively, this should depreciate the need for `TeleportFull` since all it's functionality would be merged with `Teleport`.

贡献指南

打开贡献指南

评估

这个 Issue 还没有评估数据。

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。