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

Apposition, acl and clauses #523

Closed
GPPassos opened this issue Jan 15, 2018 · 5 comments
Closed

Apposition, acl and clauses #523

GPPassos opened this issue Jan 15, 2018 · 5 comments

Comments

@GPPassos
Copy link

GPPassos commented Jan 15, 2018

In the appos page, it is stated that

While appos is normally between two nominals, there are a few cases where there is a relation with a clause, such as when describing facts or events for which appos still feels appropriate:

An example is given:

This problem , that people could lower their tax rates by choosing to become corporations , might become acute .
appos(problem-2, lower-7)

However, there is a very similar construction in the acl page:

Some languages allow finite clausal complements for nouns with a subset of nouns like fact or report. These look roughly like relative clauses, but do not have any omitted role in the dependent clause. This is the class of “content clauses” in Huddleston and Pullum 2002). These are also analyzed as acl.

the fact that nobody cares
acl(fact, cares)

Is there any significant difference between the two examples?

@jnivre
Copy link
Contributor

jnivre commented Jan 15, 2018

I would say that the first is a mistake. (I don't know who wrote it.) But the overview table of relations makes very clear that an "appos" should itself be a noun phrase, not a clause. Hence, "acl" is the right relation for this cases.

@dan-zeman
Copy link
Member

While appos is normally between two nominals, there are a few cases where there is a relation with a clause, such as when describing facts or events for which appos still feels appropriate

I would say that the first is a mistake. (I don't know who wrote it.)

Git blame says it was @manning in eed6397 :-)

@jnivre
Copy link
Contributor

jnivre commented Jan 16, 2018

I still think it is a mistake. :)

@GPPassos
Copy link
Author

Should this be closed? I also agree that this is a mistake and should be removed, but I don't know if and how this could be done (as I don't know which kinds of changes are allowed to the documentation, for instance due to versioning, etc)

martinpopel added a commit that referenced this issue Apr 26, 2018
@jnivre wrote it is "very clear that an `appos` should itself be a noun phrase, not a clause." and the example is a mistake.
Fixes #523
@martinpopel martinpopel mentioned this issue Apr 26, 2018
@martinpopel
Copy link
Member

Should this be closed?

No, I think issues like this one should be closed only after the documentation is fixed. Ideally, it should be closed by merging a PR which fixes the issue.

I don't know if and how this could be done

Each documentation page has an "edit page" button which allows to propose a change. I just used it and created a PR which deletes the mistaken example. (Anyone can open a new PR with an added sentence explaining that acl is suitable for such cases.)

which kinds of changes are allowed to the documentation

Fixing error and inconsistencies and explaining the intended guidelines is allowed anytime. Major changes (esp. backward incompatible) are allowed only with major version bump (e.g. UD1 -> UD2).

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

4 participants