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

Restore missing quotations from Dr. Sant Singh Khalsa English Translation #273

Merged
merged 1 commit into from
Feb 11, 2019

Conversation

sarabveer
Copy link
Collaborator

@sarabveer sarabveer commented Feb 10, 2019

Restore missing quotations from Dr. Sant Singh Khalsa's English Translation. Many quotations (indicating conversations or speech) were missing, most possibly an error during importing of the database.

Also removed capitalization from sentences that span multiple lines to stay close to the original translation.

These were supposed to be 2 separate commits.

Reference: English Translation of SGGS - One Volume (Dr. Sant Singh Khalsa).pdf

…tion

Many quotations (indicating conversations) were missing.
Removed captilization from sentences that span multiple lines.
@sarabveer sarabveer added Scope: Data/Sri Guru Granth Sahib Ji ○ Type Bug Regressions/production issues that do not function as intended. labels Feb 10, 2019
@sarabveer sarabveer self-assigned this Feb 10, 2019
@@ -339,7 +339,7 @@
"translations": {
"English": {
"Dr. Sant Singh Khalsa": {
"translation": "O my Lord, please save me!||1||Pause||",
"translation": "O my Lord, please save me!\" ||1||Pause||",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It’s a bit weird that quotes can span multiple lines

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep

@HarminderSingh

This comment was marked as abuse.

@bhajneet
Copy link
Member

Hi, I was looking for a correct copy of the Gurbani. I notice your work and make account. I very surprised at all work you doing, but really amazing that you confirm 3,000 lines in only 2 hrs

Your words are encouraging. If you'd like to learn more or discuss please message us using the contact info on our GitHub profile.

@sarabveer
Copy link
Collaborator Author

This was done by having a script go through a text dump of the Shabad OS Dr. Sant Singh Khalsa translation and a text dump of the PDF.

@HarminderSingh

This comment was marked as abuse.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Scope: Data/Sri Guru Granth Sahib Ji ○ Type Bug Regressions/production issues that do not function as intended.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants