huggingface / huggingface/neuralcoref

cross sentence coreference resolution

Open
#308 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
C
Stars
2.9k
Forks
467
PR merge metrics
No merged PRs in 30d

Description

Hi,

Thank you for developing this extension.
I wonder do current implementation has an option for cross sentence coreference resolution?
My workaround is adding `, ,` as sentence separator. It seems works well for many cases, but sometimes ingest the separator, so I can not map back.

For example:
**sents:** Neil Mallon Pierce Bush (born January 22, 1955) is an American businessman and investor, `, ,` He is the fourth of six children of former President George H, W, Bush and Barbara Bush (née Pierce), `, ,` His five siblings are George W, Bush, the 43rd President of the United States; Jeb Bush, a former governor of Florida; Robin Bush, died of leukemia at the age of three; Marvin; and Dorothy, `, ,` Neil Bush is currently a businessman based in Texas,
**numbe of sents before coreference resolution: 4**

**resolved_sents:** Neil Mallon Pierce Bush (born January 22, 1955) is an American businessman and investor, `, ,` Neil Mallon Pierce Bush (born January 22, 1955) is the fourth of six children of former President George H, W, Bush and Barbara Bush (née Pierce), `, ,` Neil Mallon Pierce Bush (born January 22, 1955) five siblings are George W, Bush, the 43rd President of the United States; Jeb Bush, a former governor of Florida; Barbara Bush, died of leukemia at the age of three; Marvin; and Dorothy, , Neil Mallon Pierce Bush (born January 22, 1955) Bush is currently a businessman based in Texas,
**number of sents after coreference resolution: 3**

I hope there is an existing solution I did not notice. If not, may I ask for your suggestion to fix my workaround?

Thank you

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.