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

Add s-104 and t-076 for possible <dfn> candidates in endnotes #730

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

apasel422
Copy link
Contributor

No description provided.

@apasel422 apasel422 marked this pull request as ready for review July 13, 2024 13:56
@acabal
Copy link
Member

acabal commented Jul 14, 2024

Great, thanks. I'm more interested in seeing if the xpath I proposed can be improved at all. Have you tried running it on the corpus to see what comes up? I only did a cursory investigation.

We want to see if we can craft an xpath that can return the most correct matches, with the fewest false positives. How many false positives is too many is a matter of taste, we just have to feel it out to see. The xpath I proposed on the list is just a first draft and I imagine it could be improved significantly.

@apasel422
Copy link
Contributor Author

Great, thanks. I'm more interested in seeing if the xpath I proposed can be improved at all. Have you tried running it on the corpus to see what comes up? I only did a cursory investigation.

I'll post my findings here later.

@apasel422 apasel422 marked this pull request as draft July 26, 2024 17:16
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

Successfully merging this pull request may close these issues.

2 participants