-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
UI - Higher density widget #1127
Conversation
Missing import error catch in environment detection.
Moved exceptions catch into one line.
Higher density, gives more space back to the token grid.
Different metrics shown depending on lm status.
Metrics now have hover over in frontend. Memory units are now GB instead of percentage.
Codecov ReportAttention: Patch coverage is
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #1127 +/- ##
===========================================
+ Coverage 43.18% 54.14% +10.95%
===========================================
Files 70 70
Lines 5777 5777
===========================================
+ Hits 2495 3128 +633
+ Misses 3282 2649 -633 ☔ View full report in Codecov by Sentry. |
Every metric zero'd out before metrics are sent to frontend.
Looks great. I like the compressed bar. Very nice! Code looks good too. |
MetricCard fully renamed to MetricRecord.
We're wasting too many pixels on having a separate metrics bar. Controls and metrics (which are horizontal scrollable) are now in the same bar.
Additional changes:
Shout-outs to Jingya & Nick (high-density sketches, frontend discussion)!