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

Amend category. #2364

Merged
merged 2 commits into from Aug 11, 2020
Merged

Amend category. #2364

merged 2 commits into from Aug 11, 2020

Conversation

lkhphuc
Copy link
Member

@lkhphuc lkhphuc commented Aug 10, 2020

Trong bài toán phân loại thì phân loại theo các hạng mục hay lớp đều là như nhau.
Từ này mình thấy không phải là từ khoá gì, vốn đáng ra không cần đưa vào glossary luôn. Vì có ở trong glossary nên thành ra có những PR fix (như #2329 ), với mình đang review vài chỗ thấy dịch category thành lớp đọc khá gượng gạo. Từ lớp bản thân nó mang rất nhiều nghĩa rồi, ở trường hợp này mình có từ khác thì nên dùng chứ không nên chuyển về lớp làm gì.

Nếu mọi người đồng ý thì cứ merge để từ nay về sau không phải ép category thành lớp nữa thôi, còn các phần trước đã dịch cũng không sai nên mình nghĩ cũng không cần sửa lại.

glossary.md Outdated Show resolved Hide resolved
@minhduc0711
Copy link
Contributor

okay mình đồng ý, từ này là do mình đề xuất từ lúc vẫn còn thích merge mấy từ giống nhau vào làm 1 😓

@thanhcsf thanhcsf added this to Review in progress in v0.14.x via automation Aug 11, 2020
@thanhcsf thanhcsf added this to In progress in Glossary via automation Aug 11, 2020
Copy link
Contributor

@cuongvng cuongvng left a comment

Choose a reason for hiding this comment

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

phase 2 rà lại vẫn được mà. Chứ dịch không đồng bộ em thấy không thoải mái lắm 😄

v0.14.x automation moved this from Review in progress to Reviewer approved Aug 11, 2020
@thanhcsf thanhcsf merged commit b7ae582 into master Aug 11, 2020
v0.14.x automation moved this from Reviewer approved to Done Aug 11, 2020
Glossary automation moved this from In progress to Done Aug 11, 2020
@thanhcsf thanhcsf deleted the lkhphuc-patch-2 branch August 11, 2020 12:06
@cuongvng cuongvng mentioned this pull request Aug 12, 2020
thanhcsf pushed a commit that referenced this pull request Sep 13, 2020
* Amend category.

* Update link
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Glossary
  
Done
v0.14.x
  
Done (Max)
Development

Successfully merging this pull request may close these issues.

None yet

4 participants