Skip to content

Lbliii/index page formatting#8071

Closed
lbliii wants to merge 3 commits intodetermined-ai:mainfrom
lbliii:lbliii/index-page-formatting
Closed

Lbliii/index page formatting#8071
lbliii wants to merge 3 commits intodetermined-ai:mainfrom
lbliii:lbliii/index-page-formatting

Conversation

@lbliii
Copy link
Copy Markdown
Contributor

@lbliii lbliii commented Oct 4, 2023 •
edited by tara-hpe
Loading

Moved changes to #8100

Setting as draft

Description

Test Plan

Commentary (optional)

Checklist

  • Changes have been manually QA'd
  • User-facing API changes need the "User-facing API Change" label.
  • Release notes should be added as a separate file under docs/release-notes/.
    See Release Note for details.
  • Licenses should be included for new code which was copied and/or modified from any external code.

Ticket

@lbliii lbliii requested a review from a team as a code owner October 4, 2023 18:10
@cla-bot
Copy link
Copy Markdown

cla-bot bot commented Oct 4, 2023

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have the users @lbliii on file. In order for us to review and merge your code, please start the CLA process at https://determined.ai/cla.

After we approve your CLA, we will update the contributors list (private) and comment @cla-bot[bot] check to rerun the check.

@determined-ci determined-ci added the documentation Improvements or additions to documentation label Oct 4, 2023
@determined-ci
Copy link
Copy Markdown
Collaborator

determined-ci commented Oct 4, 2023

Docsite preview being generated for this PR.
You can (eventually) find the generated docsite here.

@cla-bot
Copy link
Copy Markdown

cla-bot bot commented Oct 5, 2023

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have the users @lbliii on file. In order for us to review and merge your code, please start the CLA process at https://determined.ai/cla.

After we approve your CLA, we will update the contributors list (private) and comment @cla-bot[bot] check to rerun the check.

@determined-ci
Copy link
Copy Markdown
Collaborator

determined-ci commented Oct 5, 2023

Docsite preview being generated for this PR.
You can (eventually) find the generated docsite here.

@tara-hpe tara-hpe self-requested a review October 9, 2023 22:03
@determined-ci
Copy link
Copy Markdown
Collaborator

determined-ci commented Oct 9, 2023

Docsite preview being generated for this PR.
You can (eventually) find the generated docsite here.

tara-hpe
tara-hpe reviewed Oct 10, 2023
View reviewed changes
docs/get-started/model-examples.rst
##########
###############
Model Examples
###############
Copy link
Copy Markdown
Contributor

@tara-hpe tara-hpe Oct 10, 2023 •
edited
Loading

Choose a reason for hiding this comment

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

i decided to keep "Examples". i did not make this change in #8100

tara-hpe
tara-hpe reviewed Oct 10, 2023
View reviewed changes
docs/set-up/_index.rst
Deploy on Slurm/PBS <slurm/overview>
Cluster Configuration <cluster-configuration>
./*
./*/_index
Copy link
Copy Markdown
Contributor

@tara-hpe tara-hpe Oct 10, 2023

Choose a reason for hiding this comment

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

i decided to keep "setup-cluster" to leave room for other features in the future that may need "setup" instructions. thus "setup-cluster" indicates setup instructions for setting up (and deploying) a cluster.

@tara-hpe tara-hpe marked this pull request as draft October 13, 2023 18:23
@determined-ci
Copy link
Copy Markdown
Collaborator

determined-ci commented Oct 13, 2023

Docsite preview being generated for this PR.
You can (eventually) find the generated docsite here.

@lbliii lbliii closed this Mar 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@tara-hpe tara-hpe tara-hpe left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

documentation Improvements or additions to documentation

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@lbliii @determined-ci @tara-hpe