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

Is the current lowfat transformation including redundant clause nodes? #23

Open
ryderwishart opened this issue Jun 15, 2022 · 2 comments
Labels
question Further information is requested

Comments

@ryderwishart
Copy link
Contributor

See, for example:
Screen Shot 2022-06-15 at 9 42 14 AM

I'm not totally sure whether this is consistent behaviour that needs to be fixed or just a matter of creating specific rules for certain clause nodes.

@ryderwishart ryderwishart added the question Further information is requested label Jun 15, 2022
@jonathanrobie
Copy link
Contributor

@ryderwishart Good question. Any time a clause contains just one clause, that indicates a possible simplification. This was needed by the HPSG parser, I think.

This is one of several changes I want to make to the structure of the trees. And I want to make these changes carefully ... together ...

@rkjtan
Copy link
Contributor

rkjtan commented Jun 20, 2022

In this case, I think the different clause nodes are used to indicate the relationship between the clauses. The vocative minor clause is connected to the main clause by ClCl. The two conjoined main clauses are connected by CLaCL. Then, the subordinate clause is connected to the conjoined main clauses by sub-CL:
image

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

No branches or pull requests

3 participants