facebookresearch / facebookresearch/fairo

clean up spatial reasoning

Open
#991 0 comments 0 reactions 1 assignee Claimed by @aszlam View on GitHub
Dominant language
Jupyter Notebook
Stars
929
Forks
123
PR merge metrics
No merged PRs in 30d

Description

## Type of Issue

Select the type of issue:
- [ ] Bug report (to report a bug)
- [ ] Feature request (to request an additional feature)
- [ ] Tracker (I am just using this as a tracker)
- [x] Refactor request
- [ ] Documentation Ask

## Description

Spatial reasoning is implemented in several places in the interpreter, for example in interpret_location, interpret_reference_objects, and spatial_reasoning; relative directions except "between" for moves are handled separately in the move interpreter. These should be unified and simplified.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.