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

Update READMEs with new template for IntelPyTorch Inference/Training Optimizations #2305

Open
wants to merge 3 commits into
base: development
Choose a base branch
from

Conversation

alexsin368
Copy link
Contributor

Existing Sample Changes

Description

Update READMEs according to new template: https://github.com/oneapi-src/oneAPI-samples/wiki/AI-Getting-Started-and-Features-and-Functionality-README-Template

Fixes Issue# N/A

External Dependencies

None

Type of change

Just README update

How Has This Been Tested?

Testing not needed for README update since only formatting changed.

Copy link
Contributor

@Marina-Zubova Marina-Zubova left a comment

Choose a reason for hiding this comment

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

Thank you for updating the READMEs, I left some comments


>**Note**: Training is not performed using INT8 since using a lower precision will train a model with fewer parameters, which is likely to underfit and not generalize well.

The sample tutorial contains one Jupyter Notebook and a Python script. You can use either.

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
*Other names and brands may be claimed as the property of others. [Trademarks](https://www.intel.com/content/www/us/en/legal/trademarks.html)

Copy link
Contributor

Choose a reason for hiding this comment

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

Hi Alex @alexsin368, In this README I see that probably old version under the new one (after ---- divider). We need to remove the old README

Copy link
Contributor Author

Choose a reason for hiding this comment

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

My mistake. Just fixed it.

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.

None yet

2 participants