Skip to content

Conversation

@louie-tsai
Copy link
Contributor

Intel Xeon 6 Processors also support fp16 with auto mixed precision optimizer

Signed-off-by: Tsai, Louie <louie.tsai@intel.com>
@louie-tsai louie-tsai requested a review from a team as a code owner October 11, 2024 00:46
@github-actions
Copy link

Preview

Preview and run these notebook edits with Google Colab: Rendered notebook diffs available on ReviewNB.com.

Format and style

Use the TensorFlow docs notebook tools to format for consistent source diffs and lint for style:
$ python3 -m pip install -U --user git+https://github.com/tensorflow/docs 
$ python3 -m tensorflow_docs.tools.nbfmt notebook.ipynb
$ python3 -m tensorflow_docs.tools.nblint --arg=repo:tensorflow/docs notebook.ipynb
If commits are added to the pull request, synchronize your local branch: git pull origin graph_optimze
@8bitmp3 8bitmp3 requested review from MarkDaoust and markmcd October 11, 2024 11:56
@8bitmp3
Copy link
Contributor

8bitmp3 commented Oct 11, 2024

@MarkDaoust
Copy link
Member

Hi @louie-tsai, I don't know anything about this. Do you have any references showing that this is true?

@louie-tsai
Copy link
Contributor Author

louie-tsai commented Oct 15, 2024

@MarkDaoust
Copy link
Member

Cool, thanks. I see the processors have FP16. Does anyone know for sure that this actualy works with the Auto mixed precision optimizer?

@louie-tsai
Copy link
Contributor Author

Cool, thanks. I see the processors have FP16. Does anyone know for sure that this actualy works with the Auto mixed precision optimizer?

It works for AMP optimizer.
Loop the Intel architect @agramesh1 for more comment if any.

@agramesh1
Copy link
Contributor

Cool, thanks. I see the processors have FP16. Does anyone know for sure that this actualy works with the Auto mixed precision optimizer?

It works for AMP optimizer. Loop the Intel architect @agramesh1 for more comment if any.

Hi @MarkDaoust yes, it is now supported in TF and XLA. We have been working with @penpornk to add support for fp16 AMP using the AMX-FP16 that was introduced in the latest Intel Xeon CPUs.

@MarkDaoust MarkDaoust added the ready to pull Start merge process label Oct 18, 2024
@MarkDaoust
Copy link
Member

Cool thanks!

@louie-tsai
Copy link
Contributor Author

@markmcd
possible to help review and approve the PR?
thanks!

@markmcd
Copy link
Member

markmcd commented Oct 21, 2024

@markmcd possible to help review and approve the PR? thanks!

Done - copybara will merge shortly.

@copybara-service copybara-service bot merged commit 344f0e9 into tensorflow:master Oct 21, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

6 participants