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

Updating syseng docs and prepping for easier version updating #25

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

warthog9
Copy link
Member

No description provided.

We need a place to more easily track what version of the firmware is
current / active.  This gives us an easy place to do a pull request
against as well as pull into the docs themselves

Signed-off-by: John 'Warthog9' Hawley <[email protected]>
Signed-off-by: John 'Warthog9' Hawley <[email protected]>
This should make dealing with the versions slightly easier

Signed-off-by: John 'Warthog9' Hawley <[email protected]>
Copy link

Choose a reason for hiding this comment

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

When I view the markdown, I don't see the formatting. Is this expected because I don't have the extensions enabled, but they will be on the live site?

image

Copy link
Member Author

Choose a reason for hiding this comment

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

I'm making use of templating so if you haven't actually rendered these with sphinx, you are going to see the raw templating you are seeing.

Copy link

@milank94 milank94 left a comment

Choose a reason for hiding this comment

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

Can I suggest that we use these changes, but have them in the quickstart instead?

Let's replace the generic instruction for sending a user to the tt-kmd repo with the instruction from here on how to install kmd, firmware, etc.

@warthog9
Copy link
Member Author

warthog9 commented Dec 3, 2024

Again, the repos themselves are going to have developer centric README.md's, they aren't intended for end users - so no we don't want to point end users at the repositories themselves.

Once we have packaging the user should realistically never have to interact with the git repository itself.

End user documentation is a very different thing vs. developer centric documentation, there's a reason we wrote these docs and kept them where the end users would interact with them. If the quickstart is doing otherwise then that's going to make for a very uneven user experience, and sets a bad president for when we have proper packaging and such, which is in progress right now.

@milank94
Copy link

milank94 commented Dec 3, 2024

Again, the repos themselves are going to have developer centric README.md's, they aren't intended for end users - so no we don't want to point end users at the repositories themselves.

Once we have packaging the user should realistically never have to interact with the git repository itself.

End user documentation is a very different thing vs. developer centric documentation, there's a reason we wrote these docs and kept them where the end users would interact with them. If the quickstart is doing otherwise then that's going to make for a very uneven user experience, and sets a bad president for when we have proper packaging and such, which is in progress right now.

Aligned. My comment is to take the instructions you are proposing for this PR and move them to https://github.com/tenstorrent/tenstorrent.github.io/blob/main/core/quickstart.md. So, instead of updating syseng/softwaresetup.md, update core/quickstart.md with these instructions. This will serve as the main starting guide for new users.

The readme's within each repo remain unchanged and are developer focused only.

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