Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

word tokenizer should parse commands with multiple targets #132

Closed
ssube opened this issue Jun 20, 2021 · 0 comments
Closed

word tokenizer should parse commands with multiple targets #132

ssube opened this issue Jun 20, 2021 · 0 comments
Assignees
Labels
scope/tokenizer input tokenizer status/fixed issues that have been fixed and confirmed type/feature
Milestone

Comments

@ssube
Copy link
Owner

ssube commented Jun 20, 2021

Many new commands, such as #102 and #107, need a way to provide multiple targets: the item being used and target actor, or item being placed and target container.

Update the word tokenizer to use to, from, and a localized list of similar prepositions to split the target. Put the components in a normalized order, with the item first and actor/container second.

@ssube ssube added status/new issues that have not been addressed yet type/feature scope/tokenizer input tokenizer labels Jun 20, 2021
@ssube ssube added this to the v1.5 milestone Jun 20, 2021
@ssube ssube self-assigned this Jun 20, 2021
@ssube ssube closed this as completed in 53fc209 Jun 20, 2021
@ssube ssube added status/fixed issues that have been fixed and confirmed and removed status/new issues that have not been addressed yet labels Jun 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scope/tokenizer input tokenizer status/fixed issues that have been fixed and confirmed type/feature
Projects
None yet
Development

No branches or pull requests

1 participant