Skip to content

Documentation-updates-062022#1188

Merged
ajdecon merged 18 commits intoNVIDIA:masterfrom
tuttlebr:documentation-updates-062022
Jul 8, 2022
Merged

Documentation-updates-062022#1188
ajdecon merged 18 commits intoNVIDIA:masterfrom
tuttlebr:documentation-updates-062022

Conversation

@tuttlebr
Copy link
Copy Markdown
Contributor

Signed-off-by: Brandon Tuttle tuttlebr@ncc-1701.biz

Update documentation, partially.

tuttlebr added 2 commits June 27, 2022 13:54
Signed-off-by: Brandon Tuttle <tuttlebr@ncc-1701.biz>
Signed-off-by: Brandon Tuttle <tuttlebr@ncc-1701.biz>
ajdecon
ajdecon previously approved these changes Jun 30, 2022
Copy link
Copy Markdown
Contributor

@ajdecon ajdecon left a comment

Choose a reason for hiding this comment

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

@tuttlebr : Thank you for improving the documentation! This is awesome work and absolutely something we need 😄

I'm approving as is, but based on our offline conversation I think you said you might want to extend this further? Please let me know if you want to make changes to this PR, or if you would prefer me to merge this as-is (and make a later PR for further improvements).

@tuttlebr tuttlebr marked this pull request as draft July 8, 2022 12:19
tuttlebr added 16 commits July 8, 2022 14:35
Signed-off-by: Brandon Tuttle <tuttlebr@ncc-1701.biz>
Signed-off-by: Brandon Tuttle <tuttlebr@ncc-1701.biz>
Signed-off-by: Brandon Tuttle <tuttlebr@ncc-1701.biz>
Signed-off-by: Brandon Tuttle <tuttlebr@ncc-1701.biz>
Signed-off-by: Brandon Tuttle <tuttlebr@ncc-1701.biz>
Signed-off-by: Brandon Tuttle <tuttlebr@ncc-1701.biz>
Signed-off-by: Brandon Tuttle <tuttlebr@ncc-1701.biz>
Signed-off-by: Brandon Tuttle <tuttlebr@ncc-1701.biz>
Signed-off-by: Brandon Tuttle <tuttlebr@ncc-1701.biz>
Signed-off-by: Brandon Tuttle <tuttlebr@ncc-1701.biz>
Signed-off-by: Brandon Tuttle <tuttlebr@ncc-1701.biz>
Signed-off-by: Brandon Tuttle <tuttlebr@ncc-1701.biz>
Signed-off-by: Brandon Tuttle <tuttlebr@ncc-1701.biz>
Signed-off-by: Brandon Tuttle <tuttlebr@ncc-1701.biz>
Signed-off-by: Brandon Tuttle <tuttlebr@ncc-1701.biz>
Signed-off-by: Brandon Tuttle <tuttlebr@ncc-1701.biz>
@tuttlebr tuttlebr marked this pull request as ready for review July 8, 2022 17:26
@tuttlebr
Copy link
Copy Markdown
Contributor Author

tuttlebr commented Jul 8, 2022

This is done to the extent I was planning on modifying. It is an attempt to standardize formatting and design. I didn't check that all the docs are technically correct relative to setting up DeepOps.

Copy link
Copy Markdown
Contributor

@ajdecon ajdecon left a comment

Choose a reason for hiding this comment

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

This looks great! Thanks for putting so much effort into improving the docs. I'll go ahead and get this merged.

@ajdecon ajdecon merged commit e216065 into NVIDIA:master Jul 8, 2022
@dholt dholt mentioned this pull request Aug 23, 2022
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