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.

Sign up for GitHub

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

Jump to bottom

Session usage component in React #2803

Closed
yomybaby opened this issue Nov 1, 2024 · 0 comments · Fixed by #2814
Closed

Session usage component in React #2803

yomybaby opened this issue Nov 1, 2024 · 0 comments · Fixed by #2814
Assignees
@ironAiken2

Comments

@yomybaby
Copy link
Member

yomybaby commented Nov 1, 2024 •
edited
Loading

https://github.com/user-attachments/assets/36b58919-26e6-4069-b63e-ade66023d1e7

For the NEO session list and session detail panel(#2784), we need to implement the session usage component in React using the ComputeSessionNode fragment.
@yomybaby yomybaby changed the title Session Usage component in React Session usage component in React Nov 1, 2024
@ironAiken2 ironAiken2 mentioned this issue Nov 5, 2024
Merged
6 tasks
ironAiken2 added a commit that referenced this issue Jan 2, 2025
@ironAiken2
[FR-52] feat: Session usage monitor in react (#2814)
0900b2e 
<!--
Please precisely, concisely, and concretely describe what this PR changes, the rationale behind codes,
and how it affects the users and other developers.
-->

### This PR resolves [#2803](#2803) issue.

**Changes:**
- Displays the resources allocated to the session. Currently, the compute session node does not include a live stat field, which shows the live stat information of the main kernel associated with the session.
- Since each device has different usage, I've shown the usage below the progress bar to bring consistency to the design.

**How to test:**
- Access the Detail page of the endpoint that is currently in service.
- Verify that resource usage looks normal.

|normal|small screen|
|---|---|
|![image.png](https://graphite-user-uploaded-assets-prod.s3.amazonaws.com/lSyr8xXz1wdXALkJKzVx/b491c912-330f-4467-b995-b0d05aded2ea.png)|![image.png](https://graphite-user-uploaded-assets-prod.s3.amazonaws.com/lSyr8xXz1wdXALkJKzVx/c1bb67ea-d3c5-4b79-8ee1-18fb9e052906.png)|

**Checklist:** (if applicable)

- [x] Mention to the original issue
- [ ] Documentation
- [ ] Minium required manager version
- [ ] Specific setting for review (eg., KB link, endpoint or how to setup)
- [ ] Minimum requirements to check during review
- [ ] Test case(s) to demonstrate the difference of before/after
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ironAiken2 ironAiken2

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[FR-52] feat: Session usage monitor in react lablup/backend.ai-webui
2 participants
@yomybaby @ironAiken2