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

Fix a wrong translation, which also got the math wrong #166

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

frankdu
Copy link

@frankdu frankdu commented Mar 13, 2024

The original content was:

"Consider an example: a 5-second sound at a sampling rate of 16,000 Hz will be represented as a series of 80,000 values, while the same 5-second sound at a sampling rate of 8,000 Hz will be represented as a series of 40,000 values"

The Chinese translation was wrong because:

  1. Incomplete translation.
  2. It got the math wrong, because at 16KHz, the math is 80K data points, but not 40k.

The proposed fix is to provide full translation + get the math in shape.

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.

1 participant