-
Notifications
You must be signed in to change notification settings - Fork 349
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
docs: [FE-270] add PBS known issue - Cluster tab does not display GPU information #8719
Conversation
✅ Deploy Preview for determined-ui canceled.
|
- If the ``Cluster`` tab on the DeterminedAI WebUI does not display the GPU information, there | ||
could be an issue in the PBS configuration. Please refer to :ref:`PBS Requirements | ||
<pbs-config-requirements>` to ensure PBS is configured properly. | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- If the
Cluster
tab in the WebUI does not display the GPU information, there
may be an issue with the PBS configuration. Visit :ref:PBS Requirements <pbs-config-requirements>
to ensure PBS is properly configured.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jagadeesh545 avoid DeterminedAI
as it is reserved for talking about the company itself. Determined
is reserved for talking about the product. However, we can avoid it altogether. In the future we may also avoid WebUI
but for now it's the most valid term.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure, Tara. I updated as per your suggestions.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Requested edits
@@ -381,6 +381,10 @@ Some constraints are due to differences in behavior between Docker and Singulari | |||
PBS Known Issues | |||
****************** | |||
|
|||
- If the ``Cluster`` tab on the DeterminedAI WebUI does not display the GPU information, there |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it DeterminedAI
or Determined AI
?
Is WebUI
the appropriate wording to use?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I removed DeterminedAI
as per Tara's suggestion. I think WebUI
is fine here. Please let me know why you think it might be an inappropriate word. I can change it if needed.
@@ -381,6 +381,10 @@ Some constraints are due to differences in behavior between Docker and Singulari | |||
PBS Known Issues | |||
****************** | |||
|
|||
- If the ``Cluster`` tab on the DeterminedAI WebUI does not display the GPU information, there | |||
could be an issue in the PBS configuration. Please refer to :ref:`PBS Requirements | |||
<pbs-config-requirements>` to ensure PBS is configured properly. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Where in the PBS Requirements
(https://docs.determined.ai/latest/setup-cluster/slurm/slurm-requirements.html#pbs-requirements) would a user find the step necessary to get the GPU information displayed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PBS Requirements lists the important configuration steps necessary along with commands and sample outputs. That should help the users to ensure PBS is configured properly.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you were a user that experienced this problem, where in the PBS Requirements
would you find the solution?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated the ref link to point to the exact section for solving the issue.
@@ -381,6 +381,10 @@ Some constraints are due to differences in behavior between Docker and Singulari | |||
PBS Known Issues | |||
****************** | |||
|
|||
- If the ``Cluster`` tab on the DeterminedAI WebUI does not display the GPU information, there | |||
could be an issue in the PBS configuration. Please refer to :ref:`PBS Requirements |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we provide a little more information as to what the specific issue may be?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please let me know what kind of information you would like to be added I can add it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this update to the documentation is in response to PBS GPU issue you just worked on where if not all vnodes have the same GPUs, then it shows "", or something along those lines, right? If so, where in the documentation you are adding is the solution to that problem described?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated the ref link to point to the exact section for solving the issue.
I had been looking at the |
Sorry Rig. I updated the description with the related PR link. |
Description
Add a new PBS known issue "Cluster tab does not display GPU information" including a link to the PBS requirements section.
This is a follow up for the PR #8714
Test Plan
Tested manually
Commentary (optional)
Checklist
docs/release-notes/
.See Release Note for details.
Ticket
FE-270