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

Licensing AMRs with missing role names #11

Open
kovvalsky opened this issue Mar 5, 2018 · 1 comment
Open

Licensing AMRs with missing role names #11

kovvalsky opened this issue Mar 5, 2018 · 1 comment

Comments

@kovvalsky
Copy link

The AMRs with missing role names are accepted by Smatch and translated into triples.

(a0 / watch                               
      : (a1 / boy)                             
      :ARG1 (a2 / tv))

Triples by the smatch demo:
instance(a0,boy) ^ instance(a1,tv) ^ TOP(a0,boy) ^ ARG1(a0,a1)
This has several undesired consequences like licensing ill-formed AMRs that might get high scores.

@kovvalsky kovvalsky changed the title AMRs with missing role names Licensing AMRs with missing role names Mar 5, 2018
@snowblink14
Copy link
Owner

Adding a validation module will probably solve this issue as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants