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

Replace incorrect quotes #3833

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Replace incorrect quotes #3833

wants to merge 2 commits into from

Conversation

galcorlo
Copy link

@galcorlo galcorlo commented Oct 23, 2022

Pull Request Form

Type of Pull Request

  • Bulk fix quotation mark
  • There were lots apostrophes using double quotation mark instead of apotrophe (single quote)
  • There were many sentences with odd double quotes

Acknowledging contributors

Use single quote instead of double quote

Regex used:
 %s/^\(\c[a-zàèéíòóúüï]\)"\(\c[a-zàèéíòóúüï]\)/\1'\2/
 %s/ \(\c[a-zàèéíòóúüï]\)"\(\c[a-zàèéíòóúüï]\)/ \1'\2/g
@galcorlo galcorlo requested a review from a team as a code owner October 23, 2022 18:28
@galcorlo galcorlo requested review from data-sync-user and removed request for a team October 23, 2022 18:28
@ftyers
Copy link
Collaborator

ftyers commented Oct 26, 2022

Hi @galcorlo, have you checked to see which of these sentences already have recordings, or do none of them have recordings?

@galcorlo
Copy link
Author

Hi @ftyers, I downloaded the dataset and I could verify that at least some sentences are there. Having theses sentences with incorrect quotes is not an bloker because the person who is recording his own voice is probably able to fix it, but it's true he has to pay more attention before start to record and probably he has to read the sentence two times, as it happened to me.
Can I do something else for helping you to modify these sentences?

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.

None yet

2 participants