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 "chapter_computational-performance/hybridize_vn" - Phần 2 #497

Merged
merged 9 commits into from Feb 10, 2020

Conversation

tamnguyenvan
Copy link
Contributor

@tamnguyenvan tamnguyenvan commented Feb 5, 2020

Pull Request checklist

Đánh dấu tick (sau khi "Create Pull Request") vào các mục dưới đây

  • Pull Request này tương ứng với issue nào? Trả lời số issue sau dấu # (không có dấu cách): Close Dịch "hybridize_vn" - Phần 2 #482

  • Bản dịch này có bám sát Bảng thuật ngữ không? Nếu một từ/cụm chưa có trong Bảng thuật ngữ, hãy tạo một Pull Request riêng đề xuất cách dịch từ/cụm đó.

  • Format của bản gốc có được giữ nguyên không?

    • Các phần in nghiêng, in đậm
    • Tiêu đề (số lượng dấu # đầu dòng)
    • Bảng biểu, chú thích cho bảng (dòng phía trên mỗi bảng bắt đầu bằng dấu :)
    • Dịch các từ trong hình vẽ nếu cần. Các dòng có hình có dạng: ![caption](path)
    • Dịch các chú thích hình vẽ (thay các cụm *dịch chú thích ảnh phía trên* bằng bản dịch tiếng Việt)
    • Không dịch các phần code (nằm giữa hai dấu `````)
    • Copy các công thức toán từ bản gốc (các đoạn có $)
    • Giữ các dòng gán nhãn (bắt đầu với :label:)
    • Không tự thêm bớt các dòng trắng
  • Trong một đoạn văn, mỗi câu văn đã được viết trong một dòng, giữa các dòng không có dòng trắng. Markdown sẽ coi những dòng liền nhau không có dòng trắng là một đoạn văn. Việc này giúp công đoạn review được thuận tiện hơn.

  • Tất cả chữ trong các hình vẽ đã được dịch. Nếu chưa, vui lòng xem thêm hướng dẫn hoặc yêu cầu dịch hình.

  • Điền tên của bạn và vào mục "Những người thực hiện" ở cuối file.

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

  • Sau khi được approve, điền tên những reviewers mà bạn thấy có nhiều đóng góp cho Pull Request này.

Với reviewer

  1. Chỉ review nếu Pull Request này đã sẵn sàng cho việc review.

  2. Bạn có thể đề xuất thay đổi cách dịch mỗi dòng trực tiếp như trong hình dưới đây:
    img

Copy link
Member

@thanhcsf thanhcsf left a comment

Choose a reason for hiding this comment

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

Cảm ơn bạn về bản dịch. Mình có một vài đề xuất, bạn xem qua.

chapter_computational-performance/hybridize_vn.md Outdated Show resolved Hide resolved
chapter_computational-performance/hybridize_vn.md Outdated Show resolved Hide resolved
chapter_computational-performance/hybridize_vn.md Outdated Show resolved Hide resolved
chapter_computational-performance/hybridize_vn.md Outdated Show resolved Hide resolved
chapter_computational-performance/hybridize_vn.md Outdated Show resolved Hide resolved
chapter_computational-performance/hybridize_vn.md Outdated Show resolved Hide resolved
chapter_computational-performance/hybridize_vn.md Outdated Show resolved Hide resolved
@thanhcsf thanhcsf added this to Review in progress in v0.14.x via automation Feb 5, 2020
@thanhcsf thanhcsf added this to the 15. Computational Performance milestone Feb 5, 2020
@rootonchair
Copy link
Contributor

PR này thiếu mất phần đầu của phần 2 rồi nhé

tamnguyenvan and others added 2 commits February 7, 2020 11:23
Co-Authored-By: Thanh Doan, Steve <8481767+duythanhvn@users.noreply.github.com>
chapter_computational-performance/hybridize_vn.md Outdated Show resolved Hide resolved
chapter_computational-performance/hybridize_vn.md Outdated Show resolved Hide resolved
chapter_computational-performance/hybridize_vn.md Outdated Show resolved Hide resolved
chapter_computational-performance/hybridize_vn.md Outdated Show resolved Hide resolved
Co-Authored-By: Thanh Doan, Steve <8481767+duythanhvn@users.noreply.github.com>
@thanhcsf
Copy link
Member

thanhcsf commented Feb 7, 2020

PR này nhờ @rootonchair hoặc @lkhphuc review thêm một approve nữa rồi merge nghen.

Copy link
Member

@lkhphuc lkhphuc left a comment

Choose a reason for hiding this comment

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

Mình có mấy comments.

chapter_computational-performance/hybridize_vn.md Outdated Show resolved Hide resolved
chapter_computational-performance/hybridize_vn.md Outdated Show resolved Hide resolved
chapter_computational-performance/hybridize_vn.md Outdated Show resolved Hide resolved
chapter_computational-performance/hybridize_vn.md Outdated Show resolved Hide resolved
chapter_computational-performance/hybridize_vn.md Outdated Show resolved Hide resolved
chapter_computational-performance/hybridize_vn.md Outdated Show resolved Hide resolved
chapter_computational-performance/hybridize_vn.md Outdated Show resolved Hide resolved
Copy link
Contributor

@rootonchair rootonchair left a comment

Choose a reason for hiding this comment

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

PR sắp approve được rồi 👍

chapter_computational-performance/hybridize_vn.md Outdated Show resolved Hide resolved
@thanhcsf
Copy link
Member

thanhcsf commented Feb 8, 2020

@tamnguyenvan bạn xem qua và resolve PR này nhé.

tamnguyenvan and others added 2 commits February 9, 2020 08:21
Co-Authored-By: Phúc Lê <phuc.lkh@gmail.com>
Co-Authored-By: Thanh Doan, Steve <8481767+duythanhvn@users.noreply.github.com>
Co-Authored-By: Thanh Doan, Steve <8481767+duythanhvn@users.noreply.github.com>
Copy link
Contributor

@rootonchair rootonchair left a comment

Choose a reason for hiding this comment

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

Đây có lẽ là đợt review cuối của PR này rồi

chapter_computational-performance/hybridize_vn.md Outdated Show resolved Hide resolved
chapter_computational-performance/hybridize_vn.md Outdated Show resolved Hide resolved
@tiepvupsu
Copy link
Collaborator

@tamnguyenvan commit nốt mấy cái còn lại rồi merge nhé.

@duythanhvn tên lặp cũng không sao đâu, thiếu thì mới cần sửa. Sau revise kiểu gì mình cũng phải cleanup lại mục này.

tamnguyenvan and others added 2 commits February 10, 2020 08:30
Co-Authored-By: Thanh Doan, Steve <8481767+duythanhvn@users.noreply.github.com>
Co-Authored-By: Pham Hong Vinh <phamvinh257@gmail.com>
Co-Authored-By: Thanh Pham Chi <recca2512@gmail.com>
@tamnguyenvan
Copy link
Contributor Author

@tiepvupsu em commit rồi ạ.

v0.14.x automation moved this from Review in progress to Reviewer approved Feb 10, 2020
@tiepvupsu tiepvupsu merged commit 4c8d3e5 into master Feb 10, 2020
v0.14.x automation moved this from Reviewer approved to Done Feb 10, 2020
@tiepvupsu tiepvupsu deleted the tamnguyenvan-patch-2 branch February 10, 2020 01:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
v0.14.x
  
Done (Max)
Development

Successfully merging this pull request may close these issues.

Dịch "hybridize_vn" - Phần 2
6 participants