Skip to content

Conversation

scaliby
Copy link
Collaborator

@scaliby scaliby commented Oct 9, 2025

Fixes / Features

This is an attempt of docs refactoring performed by Gemini.

Preview: https://github.com/AI-Hypercomputer/xpk/blob/scaliby/docs-refactor/README.md

Testing / Documentation

Docs update only.

@scaliby scaliby marked this pull request as ready for review October 9, 2025 14:29
@scaliby scaliby requested a review from kzmyslona October 9, 2025 14:35
@FIoannides
Copy link
Collaborator

Can we nest CPU, GPU and Autoprovisioning links underneath Clusters? I think it makes more sense since they refer to specific "cluster create" paths/options

@scaliby
Copy link
Collaborator Author

scaliby commented Oct 10, 2025

Can we nest CPU, GPU and Autoprovisioning links underneath Clusters? I think it makes more sense since they refer to specific "cluster create" paths/options

Added nesting.

@scaliby scaliby requested a review from jamOne- October 10, 2025 12:36
Copy link
Collaborator

@jamOne- jamOne- left a comment

Choose a reason for hiding this comment

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

LGTM, but please wait for approvals from Feidias and Kasia.

XPK supports a variety of hardware accelerators.
| Accelerator | Type | Create Cluster | Create Workload |
|-------------|--------------------|---------------------------------------------------------------------------------------------------------------|-----------------------------------------------------------------------------------------------------------------|
| TPU | v4 | [docs](./docs/usage/clusters.md) | [docs](./docs/usage/workloads.md) |
Copy link
Collaborator

Choose a reason for hiding this comment

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

One thing that I personally find a bit confusing, is that all TPU docs lead to the same page, so as a user if I click on the link next to the TPU v4 row, I expect to get instructions for TPU v4, but instead all are fixed to TPU v5 and the user may not notice that. Similarly for GPUs and H100. Not sure how we can fix that, I guess if there is some way to use url params it could work, simpler would probably be to go back to the list format that we had before and have general links for cluster and workload create.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

There is no docs specific to TPU, so that's why I link to generic ones. I think this is a good starting point and good organization method. We can later address by adding docs specific for TPU. IMO table is much cleaner than the list present before.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants