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 "kernel" #1448

Merged
merged 2 commits into from May 14, 2020
Merged

Amend "kernel" #1448

merged 2 commits into from May 14, 2020

Conversation

cuongvng
Copy link
Contributor

@cuongvng cuongvng commented May 12, 2020

Review thấy có bạn dịch kernel methodhàm bộ lọc sợ quá nên đề xuất thêm vào glossary.

glossary.md Outdated Show resolved Hide resolved
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ác từ đề xuất này có thật sự cần đưa vào Bảng thuật ngữ không các bạn nhỉ?

@cuongvng
Copy link
Contributor Author

@duythanhvn các từ này có vấn đề gì vậy anh?

@lkhphuc
Copy link
Member

lkhphuc commented May 12, 2020

Mình nghĩ là cần đó @duythanhvn . từ kernel này bị overload nghĩa rất nhiều, như @cuongvng có nói có bạn nhầm sang nhân bộ lọc của CNN.

@thanhcsf
Copy link
Member

@lkhphuc Yepp, mình đồng tình chỗ kernel dễ overload về nghĩa. Vấn đề là cần bổ sung context, nếu để nguyên như ban đầu thêm vào mình thấy có không cần thiết, kernel đã được bổ sung trong bảng thuật ngữ, Function vẫn làm hàm, Method vẫn là phương pháp..

Cái em Cường để trên mình thấy hên xui lắm mới có anh em dịch ra như thế, pha bẻ lái đấy gắt phết..

@cuongvng
Copy link
Contributor Author

ở PR #1363 em thấy bỏ định nghĩa kernel trong CNN là bộ lọc, chỉ còn 1 nghĩa rất chung chung là nhân. Em đọc conversation cũng chưa hiểu bỏ đi để làm gì lắm, trước giờ vẫn dịch trong context CNN là bộ lọc mà?

@minhduc0711
Copy link
Contributor

minhduc0711 commented May 13, 2020

@cuongvng sau đó thì mọi người lại muốn tách vì muốn từ ngữ đa dạng.
Thực ra thảo luận bắt đầu từ đây

glossary.md Outdated Show resolved Hide resolved
Co-authored-by: Duc Pham <phamminhduc0711@gmail.com>
@cuongvng cuongvng changed the title Add "kernel method", "kernel function" Amend "kernel" May 14, 2020
@thanhcsf thanhcsf added this to Review in progress in v0.14.x via automation May 14, 2020
@thanhcsf thanhcsf added this to In progress in Glossary via automation May 14, 2020
v0.14.x automation moved this from Review in progress to Reviewer approved May 14, 2020
@thanhcsf thanhcsf merged commit 366c369 into d2l-ai:master May 14, 2020
v0.14.x automation moved this from Reviewer approved to Done May 14, 2020
Glossary automation moved this from In progress to Done May 14, 2020
thanhcsf pushed a commit that referenced this pull request Sep 13, 2020
* Add "kernel method", "kernel function"

* Update glossary.md

Co-authored-by: Duc Pham <phamminhduc0711@gmail.com>

Co-authored-by: Duc Pham <phamminhduc0711@gmail.com>
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