Skip to content

Readme enhancement for Ironwood(tpu recipe)#157

Closed
sivanishwanthm wants to merge 4 commits into
AI-Hypercomputer:mainfrom
sivanishwanthm:Ironwood-Deepseek-448
Closed

Readme enhancement for Ironwood(tpu recipe)#157
sivanishwanthm wants to merge 4 commits into
AI-Hypercomputer:mainfrom
sivanishwanthm:Ironwood-Deepseek-448

Conversation

@sivanishwanthm
Copy link
Copy Markdown
Contributor

Summary Updated README for Ironwood XPK recipe to improve User Experience.

Changes

Get the Recipe: Added the "Get the Recipe" block to the READMEs.

Pretraining Workload: Added the nano command instruction to the workload block.

Monitor the Job: Updated the monitoring command to kubectl logs -f ${POD_NAME} for accuracy.

@@ -247,6 +254,7 @@ To run the benchmark, first make the script executable and then run it:
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Let's update this comment to mention editing and what needs to be edited

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Thanks for the feedback! I've updated the changes in the Pretraining Workload Block.

```bash
kubectl get jobset -n default ${WORKLOAD_NAME}
kubectl logs -f -n default jobset/${WORKLOAD_NAME}-0-worker-0
kubectl logs -f ${PDD_NAME}
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

If this is meant to be POD_NAME , how are we telling user to set this ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Updated. I fixed the typo and changed the instructions so the user explicitly lists the pods to find the correct <POD_NAME> before running the logs command

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