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] add fp16 training #7

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

[add] add fp16 training #7

wants to merge 1 commit into from

Conversation

Muyun99
Copy link

@Muyun99 Muyun99 commented Jul 6, 2022

Hello, chuanguang

It`s a PR about fp16 training. I use this code to run your CIRKD on four RTX 3090 without hyperparameter changes. The performance is still aligned with faster training speeds and reduced GPU memory.

I implemented fp16 in a straightforward way with if-else style. If you have any suggestions on code style please contact me. BTW, I just started my internship at Horizon, you can find me on feishu.

Looking forward to your reply. Thanks!

Best Regards!
Yun

@winycg
Copy link
Owner

winycg commented Jul 7, 2022

Hello Yu
Thanks for your great contributions! I am very glad to merge your fp16 training code to this codebase. It seems that the added code lengthens the original training files and may impact the simplicity of the core code. I hope you can create a single training file to contain your fp16 training code, such as train_kd_fp16.py. Thanks for your contributions again!

Best Regards!
Chuanguang

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants