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.

Sign up for GitHub

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

Jump to bottom

translation: Update chapter_introduction/algorithms_are_everywhere.md #972

Merged
merged 3 commits into from
Dec 21, 2023
Merged

translation: Update chapter_introduction/algorithms_are_everywhere.md #972

merged 3 commits into from
Dec 21, 2023

Conversation

K3v123
Copy link
Contributor

@K3v123 K3v123 commented Nov 29, 2023

changed or refined parts of the words and sentences including tips. Some of them I didnt change that much because im worried that it might not meet the requirement of accuracy. some other ones i changed a lot to make it sound better, but also kind of following the same wording as the CN version

If this PR is related to coding or code translation, please fill out the checklist and paste the console outputs to the PR.

  • I've tested the code and ensured the outputs are the same as the outputs of reference code.
  • I've checked the code (formatting, comments, indentation, file header, etc) carefully.
  • The code does not rely on a particular environment or IDE and can be executed on a standard system (Win, macOS, Ubuntu).

@K3v123
Update algorithms_are_everywhere.md
345d34e 
changed or refined parts of the words and sentences including tips.
Some of them I didnt change that much because im worried that it might not meet the requirement of accuracy. 
some other ones i changed a lot to make it sound better, but also kind of following the same wording as the CN version
@krahets krahets changed the title Update algorithms_are_everywhere.md translation: Update chapter_introduction/algorithms_are_everywhere.md Nov 29, 2023
@krahets krahets added translation English translation documents documents-related labels Nov 29, 2023
@krahets
Copy link
Owner

krahets commented Nov 29, 2023 •
edited
Loading

Hi @K3v123 , thanks for the PR! I've invited you to the Project.

The PR will be merged after approval by 3 reviewers.

@krahets
Copy link
Owner

krahets commented Nov 29, 2023

Join us on Discord

K3v123 and others added 2 commits December 17, 2023 15:29
@K3v123
Update algorithms_are_everywhere.md
a388581 
re-edited the dictionary part from Piyin to just normal Eng dictionary. 
again thank you very much hpstory for you suggestion.
@krahets
Update algorithms_are_everywhere.md
2a9745f
krahets
krahets approved these changes Dec 21, 2023
View reviewed changes
Copy link
Owner

@krahets krahets left a comment

Choose a reason for hiding this comment

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

Thanks for the PR! @K3v123

@krahets krahets merged commit 4204130 into krahets:en Dec 21, 2023
1 check passed
krahets added a commit that referenced this pull request Dec 21, 2023
@krahets @yuelinxin
@Phoenix0415 @longsizhuo @QiLOL @thomasq0 @K3v123 @yanedie
translation: English Translation of the chapter of preface(part), int…
46813a5 
…roduction and complexity analysis(part) (#994)

* Translate 1.0.0b6 release with the machine learning translator.

* Update Dockerfile
A few translation improvements.

* Fix a badge logo.

* Fix EN translation of chapter_appendix/terminology.md (#913)

* Update README.md

* Update README.md

* translation: Refined the automated translation of README (#932)

* refined the automated translation of README

* Update index.md

* Update mkdocs-en.yml

---------

Co-authored-by: Yudong Jin <[email protected]>

* translate: Embellish chapter_computational_complexity/index.md (#940)

* translation: Update chapter_computational_complexity/performance_evaluation.md (#943)

* Update performance_evaluation.md

* Update performance_evaluation.md

* Update performance_evaluation.md

change 'methods' to 'approaches' on line 15

* Update performance_evaluation.md

on line 21, change the sentence to 'the results could be the opposite on another computer with different specifications.'

* Update performance_evaluation.md

delete two short sentence on line 5 and 6

* Update performance_evaluation.md

change `unavoidable` to `inevitable` on line 48

* Update performance_evaluation.md

small changes on line 23

* translation: Update terminology and improve readability in preface summary (#954)

* Update terminology and improve readability in preface summary

This commit made a few adjustments in the 'summary.md' file for clearer and more accessible language. "Brushing tool library" was replaced with "Coding Toolkit" to better reflect common terminology. Also, advice for beginners in algorithm learning journey was reformulated to imply a more positive approach avoiding detours and common pitfalls. The section related to the discussion forum was rewritten to sound more inviting to readers.

* Format

* Optimize the translation of
chapter_introduction/algorithms_are_everywhere.

* Add .gitignore to Java subfolder.

* Update the button assets.

* Fix the callout

* translation: chapter_computational_complexity/summary to en (#953)

* translate chapter_computational_complexity/summary

* minor format

* Update summary.md with comment

* Update summary.md

* Update summary.md

* translation: chapter_introduction/what_is_dsa.md (#962)

* Optimize translation of what_is_dsa.md

* Update

* translation: chapter_introduction/summary.md (#963)

* Translate chapter_introduction/summary.md

* Update

* translation: Update README.md (#964)

* Update en translation of README.md

* Update README.md

* translation: update space_complexity.md (#970)

* update space_complexity.md

* the rest of translation piece

* Update space_complexity.md

---------

Co-authored-by: ThomasQiu <[email protected]>
Co-authored-by: Yudong Jin <[email protected]>

* translation: Update chapter_introduction/index.md (#971)

* Update index.md

sorry, first time doing this... now this is the final change.
changes:
title of the chapter is shorter.
refined the abstract.

* Update index.md

---------

Co-authored-by: Yudong Jin <[email protected]>

* translation: Update chapter_data_structure/classification_of_data_structure.md (#980)

* update classification_of_data_structure.md

* Update classification_of_data_structure.md

---------

Co-authored-by: Yudong Jin <[email protected]>

* translation: Update chapter_introduction/algorithms_are_everywhere.md (#972)

* Update algorithms_are_everywhere.md

changed or refined parts of the words and sentences including tips.
Some of them I didnt change that much because im worried that it might not meet the requirement of accuracy. 
some other ones i changed a lot to make it sound better, but also kind of following the same wording as the CN version

* Update algorithms_are_everywhere.md

re-edited the dictionary part from Piyin to just normal Eng dictionary. 
again thank you very much hpstory for you suggestion.

* Update algorithms_are_everywhere.md

---------

Co-authored-by: Yudong Jin <[email protected]>

* Prepare merging into main branch.

* Update buttons

* Update Dockerfile

* Update index.md

* Update index.md

* Update README

* Fix index.md

* Fix mkdocs-en.yml

---------

Co-authored-by: Yuelin Xin <[email protected]>
Co-authored-by: Phoenix Xie <[email protected]>
Co-authored-by: Sizhuo Long <[email protected]>
Co-authored-by: Spark <[email protected]>
Co-authored-by: Thomas <[email protected]>
Co-authored-by: ThomasQiu <[email protected]>
Co-authored-by: K3v123 <[email protected]>
Co-authored-by: Jin <[email protected]>
yuelinxin
yuelinxin reviewed Dec 22, 2023
View reviewed changes
Copy link
Contributor

@yuelinxin yuelinxin left a comment

Choose a reason for hiding this comment

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

Sorry for not commenting on this before its merge, but please open a new PR to address the mentioned issues as needed, thank you! @K3v123

docs-en/chapter_introduction/algorithms_are_everywhere.md
@@ -4,11 +4,11 @@ When we hear the word "algorithm", we naturally think of mathematics. However, m

Before we formally discuss algorithms, an interesting fact is worth sharing: **you have already learned many algorithms unconsciously and have become accustomed to applying them in your daily life**. Below, I will give a few specific examples to prove this point.

**Example 1: Looking Up a Dictionary**. In a dictionary, each Chinese character corresponds to a pinyin, and the dictionary is arranged in alphabetical order of pinyin. Suppose we need to find a character whose pinyin starts with the letter $r$. This is usually achieved in the following way:
**Example 1: Looking Up a Dictionary**. In a standard dictionary, each word corresponds to a phonetic transcription and the dictionary is organized alphabetically based on these transcriptions. Let's say we're looking for a word that begins with the letter $r$. This is typically done in the following way:
Copy link
Contributor

Choose a reason for hiding this comment

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

In the case of the difference between different languages, there's no need to stick that close to the original text, simply stating that in an English dictionary, the words are listed alphabetically is sufficient. Adding this "phonetic transcription" bit just adds more confusion.

docs-en/chapter_introduction/algorithms_are_everywhere.md
2. Since $r$ comes after $m$ in the pinyin alphabet, we exclude the first half of the dictionary and narrow the search to the second half.
3. Repeat steps `1.` and `2.` until you find the page where the pinyin starts with $r$.
1. Open the dictionary around its midpoint and note the first letter on that page, assuming it to be $m$.
2. Given the sequence of words following the initial letter $m$, estimate where words starting with the letter $r$ might be located within the alphabetical order.
Copy link
Contributor

Choose a reason for hiding this comment

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

I think we should stick to the original reasoning here, it makes more sense.
"Since $r$ comes after $m$ alphabetically, we exclude the first half of the dictionary and narrow the search to the second half."

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yuelinxin yuelinxin yuelinxin left review comments

@hpstory hpstory hpstory left review comments

@krahets krahets krahets approved these changes

Assignees

@K3v123 K3v123

Labels
documents documents-related translation English translation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@K3v123 @krahets @hpstory @yuelinxin