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

Revise Chương 24-27 #431

Merged
merged 15 commits into from
Dec 11, 2019
Merged

Revise Chương 24-27 #431

merged 15 commits into from
Dec 11, 2019

Conversation

ngcthuong
Copy link
Contributor

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 # Revise chương 24-27 #416

  • Bạn có kiến thức về Học Máy không? Chúng tôi ưu tiên những người có trước kiến thức về Học Máy để 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.

  • Pull request này đã sẵn sàng cho việc review chưa?

Lưu ý

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

@ngcthuong ngcthuong changed the title 1st draft Revise Chương 24-27 Dec 9, 2019
@ngcthuong ngcthuong changed the title Revise Chương 24-27 [WIP] Revise Chương 24-27 Dec 9, 2019
@ngcthuong ngcthuong changed the title [WIP] Revise Chương 24-27 Revise Chương 24-27 Dec 11, 2019
@ngcthuong ngcthuong added the review wanted Pull Request cần được review sớm label Dec 11, 2019
ngcthuong and others added 11 commits December 11, 2019 16:52
Co-Authored-By: tiepvupsu <[email protected]>
Co-Authored-By: tiepvupsu <[email protected]>
Co-Authored-By: tiepvupsu <[email protected]>
Co-Authored-By: tiepvupsu <[email protected]>
Co-Authored-By: tiepvupsu <[email protected]>
Co-Authored-By: tiepvupsu <[email protected]>
Co-Authored-By: tiepvupsu <[email protected]>
Co-Authored-By: tiepvupsu <[email protected]>
Co-Authored-By: tiepvupsu <[email protected]>
Co-Authored-By: tiepvupsu <[email protected]>
Co-Authored-By: tiepvupsu <[email protected]>
@tiepvupsu tiepvupsu merged commit 4e5d04d into master Dec 11, 2019
@tiepvupsu tiepvupsu deleted the Revise24-27 branch December 11, 2019 08:00
Copy link
Contributor

@william-vu william-vu left a comment

Choose a reason for hiding this comment

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

E thấy ok, chỉ có 1 comment nhỏ về tiêu đề mới của chap 24 và chap 25.

@@ -1,27 +1,27 @@
> # 24. Bias vs. Variance tradeoff

# 24. Sự đánh đổi giữa Độ chệch và Phương sai
# 24. Đánh đổi giữa Độ chệch và Phương sai
Copy link
Contributor

Choose a reason for hiding this comment

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

trong tiêu đề không có chữ "sự" trong khi câu bên dưới lại có. Có sợ không nhất quán ko bác?
Same comment cho tiêu đề của Chap 25.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Không cần chữ sự trong tiêu đề cũng được. Mà mình chủ đích bỏ từ "sự" rồi mà nhỉ?

BTW, mình tính có thêm PR liên quan tới sửa tiêu đề từng chương nữa.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
review wanted Pull Request cần được review sớm
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants