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

Add "warmup" #2008

Merged
merged 3 commits into from Jul 7, 2020
Merged

Add "warmup" #2008

merged 3 commits into from Jul 7, 2020

Conversation

davidnvq
Copy link
Contributor

@davidnvq davidnvq commented Jul 5, 2020

Từ warmup này xuất hiện trong phần Learning rate scheduler.
Mình tạm đề xuất là khởi động, mượn bên thể thao. Mà vẫn phân vân vì ngày xưa đi trường hè, có thầy Hưng (director VinAI giờ) có trình bày về khởi đầu nguội, rồi có 1 cụm từ làm nóng gì đó. Nhớ mang máng vậy.

@thanhcsf thanhcsf added this to Review in progress in v0.14.x via automation Jul 5, 2020
@thanhcsf thanhcsf added this to In progress in Glossary via automation Jul 5, 2020
@thanhcsf thanhcsf added this to the 11. Optimization Algorithms milestone Jul 5, 2020
@davidnvq davidnvq mentioned this pull request Jul 5, 2020
6 tasks
Copy link
Contributor

@minhduc0711 minhduc0711 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 thấy khởi động cũng được rồi

v0.14.x automation moved this from Review in progress to Reviewer approved Jul 5, 2020
@thanhcsf
Copy link
Member

thanhcsf commented Jul 5, 2020

Cái chỗ ngữ cảnh bổ sung đấy mình đang thấy Bảng thuật ngữ sử dụng cả Anh, Việt hơi lẫn lộn. Anh em muốn giữ như vậy, hay chọn một ngôn ngữ để bổ sung ngữ cảnh thôi; hoặc tiếng Anh, hoặc tiếng Việt?

cc: @minhduc0711 @davidnvq @lkhphuc @rootonchair @cuongvng

@cuongvng
Copy link
Contributor

cuongvng commented Jul 5, 2020

em vote ngữ cảnh tiếng Việt.

@minhduc0711
Copy link
Contributor

@duythanhvn Em thì thích tiếng anh hơn, tại đang có sẵn mấy ngữ cảnh như CNN, softmax.

Ngoài ra thì em thấy dấu ngoặc đơn đang hơi bị overload: ngữ cảnh, cách viết tắt, từ optional, loại từ.
Em nghĩ mình nên thống nhất 1 cách format gọn gàng hơn chút (có thể thêm cả cột mới cho context?).

@davidnvq
Copy link
Contributor Author

davidnvq commented Jul 5, 2020

Thế chắc bổ sung thêm cột Note nữa r điền nhưng thông tin bổ sung đó vào 1 cột phát sinh đó

@thanhcsf
Copy link
Member

thanhcsf commented Jul 6, 2020

Thế chắc bổ sung thêm cột Note nữa r điền nhưng thông tin bổ sung đó vào 1 cột phát sinh đó

Ngôn ngữ cho phần ngữ cảnh ông giáo muốn là gì? Ông giáo vote tiếng Anh hay tiếng Việt?

@lkhphuc
Copy link
Member

lkhphuc commented Jul 6, 2020

Thế chắc bổ sung thêm cột Note nữa r điền nhưng thông tin bổ sung đó vào 1 cột phát sinh đó

Ngôn ngữ cho phần ngữ cảnh ông giáo muốn là gì? Ông giáo vote tiếng Anh hay tiếng Việt?

Mình thấy cột tiếng nào thì chú thích bằng tiếng đấy thôi, như chú thích cho từ warm-up thì mình nên thêm bằng tiếng anh, còn chú thích cho cách dịch tiếng việt thì mình mở ngoặc tiếng việt bên phần cách dịch.

@thanhcsf
Copy link
Member

thanhcsf commented Jul 6, 2020

Mình thấy cột tiếng nào thì chú thích bằng tiếng đấy thôi, như chú thích cho từ warm-up thì mình nên thêm bằng tiếng anh, còn chú thích cho cách dịch tiếng việt thì mình mở ngoặc tiếng việt bên phần cách dịch.

Đề xuất này khá hợp lý. Anh em thấy sao?

glossary.md Outdated
@@ -484,6 +484,7 @@ Nếu bạn cho rằng một từ không nên dịch ra tiếng Việt, bạn c
## W
| English | Tiếng Việt | Thảo luận tại |
|-------------------------------------------|-------------------|----------------------------------------------|
| warmup (trong định thời tốc độ học) | khởi động | [https://git.io/JJIT5](https://git.io/JJIT5) |
Copy link
Contributor

Choose a reason for hiding this comment

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

em đồng ý.

Suggested change
| warmup (trong định thời tốc độ học) | khởi động | [https://git.io/JJIT5](https://git.io/JJIT5) |
| warmup (in learning rate scheduling) | khởi động (trong định thời tốc độ học) | [https://git.io/JJIT5](https://git.io/JJIT5) |

Copy link
Member

Choose a reason for hiding this comment

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

Mình thấy phần (trong định thời tốc độ học) chỗ này hơi thừa, không cần thiết lắm. Chú thích bên tiếng việt giống như kiểu (câu) truy vấn ấy. Mình mở ngoặc để chú thích linh động nhiều cách dịch, có hay không từ câu cũng được.

Copy link
Contributor Author

@davidnvq davidnvq Jul 7, 2020

Choose a reason for hiding this comment

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

Thế sửa lại như sau nhé :3

Suggested change
| warmup (trong định thời tốc độ học) | khởi động | [https://git.io/JJIT5](https://git.io/JJIT5) |
| warmup (in learning rate scheduling) | khởi động | [https://git.io/JJIT5](https://git.io/JJIT5) |

Copy link
Contributor

Choose a reason for hiding this comment

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

ủa em hiểu là đang thảo luận thêm chú thích ngữ cảnh như nào, chứ không phải chú thích cách dịch.

@davidnvq davidnvq merged commit 47cd923 into master Jul 7, 2020
v0.14.x automation moved this from Reviewer approved to Done Jul 7, 2020
Glossary automation moved this from In progress to Done Jul 7, 2020
@davidnvq davidnvq deleted the davidnvq-glossary-warmup branch July 7, 2020 02:03
@thanhcsf
Copy link
Member

thanhcsf commented Jul 7, 2020

Hic, ông giáo @davidnvq bình tĩnh nhé, này chưa xong mà..

@davidnvq
Copy link
Contributor Author

davidnvq commented Jul 7, 2020

À okie, hiểu nhầm là đôi chút :'(

@minhduc0711
Copy link
Contributor

@duythanhvn thế cuối cùng là sửa hết về tiếng anh ạ?

@thanhcsf
Copy link
Member

thanhcsf commented Jul 7, 2020

Ừa, tạm thời thì đưa về tiếng Anh hết hen.

thanhcsf pushed a commit that referenced this pull request Sep 13, 2020
* Update glossary.md

* fix order + add link

* Update glossary.md

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

5 participants