Skip to content

Update IntelPython_daal4py_DistributedKMeans readme #1410

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

Merged
merged 1 commit into from
Mar 8, 2023
Merged

Update IntelPython_daal4py_DistributedKMeans readme #1410

merged 1 commit into from
Mar 8, 2023

Conversation

jkinsky
Copy link
Contributor

@jkinsky jkinsky commented Mar 7, 2023

Restructured to match the new readme template—more or less. Changed sample name to match the name in sample.json file. Restructured sections to increase clarity. Clarified information on running sample in devcloud. Rewrote some text to improve clarity. Updated branding based on names in database.

Existing Sample Changes

Description

N/A

Fixes Issue#

External Dependencies

List any external dependencies created as a result of this change.

Type of change

Please delete options that are not relevant. Add a 'X' to the one that is applicable.

  • Documentation updat

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • VSCode

Restructured to match the new readme template—more or less. Changed sample name to match the name in sample.json file. Restructured sections to increase clarity. Clarified information on running sample in devcloud. Rewrote some text to improve clarity. Updated branding based on names in database.
@jkinsky jkinsky requested a review from ishaghosh27 March 7, 2023 20:24
@jkinsky
Copy link
Contributor Author

jkinsky commented Mar 7, 2023

Adding @jimmytwei for notices.

@jimmytwei jimmytwei merged commit 3a8529c into oneapi-src:2023.1.1_AIKit Mar 8, 2023
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.

3 participants