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

Dịch chương 32 #251

Merged
merged 10 commits into from Nov 8, 2019
Merged

Dịch chương 32 #251

merged 10 commits into from Nov 8, 2019

Conversation

william-vu
Copy link
Contributor

first commit

Nếu bạn tạo Pull Request để dịch, vui lòng hoàn thành các thông tin dưới đây.

  • Đánh dấu [x] (không có dấu cách quanh x) thể hiện dấu tick vào các câu dưới đây cho các câu trả lời 'Có' hoặc 'Đúng'

  • Pull Request này được link tới một issue? Đánh số issue đó ngay sau dấu #

  • Bạn có kiến thức về Machine Learning không? Chúng tôi ưu tiên những người có trước kiến thức về Machine Learning để việc review được thuận tiện hơn.

  • Bạn đã đọc Bảng thuật ngữ chưa? Bạn cần dịch các thuật ngữ theo bảng thuật ngữ chung.

  • Tất cả các thuật ngữ trong chương này đã có trong Bảng thuật ngữ chưa? Nếu chưa, bạn có thể tạo một issue hoặc pull request mới để bổ sung nghĩa của từ đó vào Bảng thuật ngữ.

  • Pull request này đã sẵn sàng cho việc review chưa? Nếu đã sẵn sàng, bạn vui lòng gán label 'review wanted' cho pull request này. Nếu chưa, bạn vui lòng thêm cụm "[WIP]" (work in progress) vào trước tiêu đề của pull request.

Lưu ý

Chỉ bắt đầu review khi mọi mục trên đây đều đã được tick.

first commit
@william-vu william-vu added the review wanted Pull Request cần được review sớm label Nov 4, 2019
@tiepvupsu tiepvupsu mentioned this pull request Nov 4, 2019
6 tasks
chapters/ch32.md Outdated Show resolved Hide resolved
chapters/ch32.md Outdated Show resolved Hide resolved
chapters/ch32.md Outdated Show resolved Hide resolved
chapters/ch32.md Outdated Show resolved Hide resolved
chapters/ch32.md Outdated Show resolved Hide resolved
@tiepvupsu tiepvupsu changed the title Dịch chap32.md Dịch chương 32 Nov 5, 2019
address comments from 1612628 and sonvx
@tiepvupsu tiepvupsu added the high priority Cần được hoàn thành sớm label Nov 6, 2019
chapters/ch32.md Outdated Show resolved Hide resolved
chapters/ch32.md Outdated Show resolved Hide resolved
address comments from Tiep and naml3i
chapters/ch32.md Outdated Show resolved Hide resolved
chapters/ch32.md Outdated Show resolved Hide resolved
update theo comments của SummoBBQ
chapters/ch32.md Outdated Show resolved Hide resolved
chapters/ch32.md Outdated Show resolved Hide resolved
@william-vu
Copy link
Contributor Author

@SumoBBQ : nếu bạn comment vào cái conversation đã được resolved thì unresolved giúp mình nhé; mình thấy email báo có comments mới của bạn mà tìm ở đây mãi chưa thấy (cái đoạn gì cấp số cộng ấy).

Copy link
Contributor

@ngcthuong ngcthuong left a comment

Choose a reason for hiding this comment

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

Bản dịch khá ổn. Mình chỉ có một số góp ý nhỏ.

chapters/ch32.md Outdated Show resolved Hide resolved
chapters/ch32.md Outdated Show resolved Hide resolved
chapters/ch32.md Outdated Show resolved Hide resolved
chapters/ch32.md Outdated Show resolved Hide resolved
chapters/ch32.md Outdated Show resolved Hide resolved
chapters/ch32.md Show resolved Hide resolved
address Thuong's comments
@william-vu
Copy link
Contributor Author

@ngcthuong : sửa theo comments của cậu rồi nhé. :D

address per Tiep's comments
@tiepvupsu tiepvupsu merged commit 31f71eb into master Nov 8, 2019
@thanhcsf thanhcsf deleted the william-vu-chap32 branch November 11, 2020 03:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
high priority Cần được hoàn thành sớm review wanted Pull Request cần được review sớm
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants