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

Update evaluation.mdx #129

Closed
wants to merge 1 commit into from
Closed

Conversation

practice-dump
Copy link
Contributor

According to the definition given of Insertion and deletion operation, changed content a bit

Substitutions (S): where we transcribe the wrong word in our prediction (“sit” instead of “sat”)
Insertions (I): where we add an extra word in our prediction
Deletions (D): where we remove a word in our prediction
(Taken from course page)

So here it is mentioned we perform operation on prediction. So if "mat" is not predicted by the model, it should be inserted not deleted right?

According to the definition given of Insertion and deletion operation, changed
@HuggingFaceDocBuilderDev
Copy link

HuggingFaceDocBuilderDev commented Aug 29, 2023

The documentation is not available anymore as the PR was closed or merged.

@MKhalusova
Copy link
Contributor

The original description is correct. The substitutions, insertions, and deletions are types of errors that we observe (e.g. there's no "mat", it's missing, aka "deleted"), not actions that need to be taken to fix errors.

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.

3 participants