Variable Importance chart: relative importance, fix label truncation, resizable sidebar#6
Merged
jrkasprzyk merged 3 commits intomainfrom May 7, 2026
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
… resizable sidebar Agent-Logs-Url: https://github.com/jrkasprzyk/ArborView/sessions/f1f36884-098f-4d04-a9ef-f8014ec8008b Co-authored-by: jrkasprzyk <4161712+jrkasprzyk@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Update variable importance chart with relative importance and resizing
Variable Importance chart: relative importance, fix label truncation, resizable sidebar
May 5, 2026
zaca1902
reviewed
May 7, 2026
Collaborator
zaca1902
left a comment
zaca1902
left a comment
There was a problem hiding this comment.
Everything here looks good to me! Once I get my Vercel account working (it is currently running into authentication issues), I can also double-check the test deployment from this branch.
Combine tab/panel layout from main with resize-handle from PR branch. Resize handle placed between canvas section and sidebar, preserving both features.
jrkasprzyk
approved these changes
May 7, 2026
Owner
jrkasprzyk
left a comment
jrkasprzyk
left a comment
There was a problem hiding this comment.
Everything looks good here.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The Variable Importance panel showed raw rpart scores (not meaningful to end users), truncated long variable names at 110 px, and had no way to resize the panel.
Changes
Relative importance display
renderImportance()now shows each variable's share of total importance as a percentage (val / Σall * 100) rather than the raw rpart scoreFix label truncation
110px 1fr 50px) with a 2-row stacked layout: name + percentage on row 1, full-width bar on row 2cumulative_EndOf_WY3are now fully visible; ellipsis only triggers for extreme lengthsResizable sidebar
<div id="resize-handle">drag handle between canvas and sidebar in the HTML grid1fr 5px var(--sidebar-w)— sidebar width driven by a CSS custom property (--sidebar-w: 340px)initResizeHandle()handlesmousedown/mousemove/mouseup, clamping width to[220, 640] pxDEFAULT_SIDEBAR_W = 340constant keeps the JS fallback in sync with the CSS default