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.

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

changes for frontend of h2o flow to enable TreeVisualization #11

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

msrimat
Copy link

@msrimat msrimat commented Aug 13, 2016

screen shot 2016-08-12 at 5 12 31 pm

@micahstubbs
Copy link
Contributor

will add these comments

screen shot 2016-12-09 at 1 10 22 pm

@micahstubbs
Copy link
Contributor

another comment from slack

oh @jfohl - sorry, was a question for @Leland. just wondering if anyone is working on the treevis. The answer was: 1) @micah is working on reviewing the flow PR, but blocked on backend changes in H2O to get the tree data, 2) I mentioned there’s a tree interpreter I saw Tom made, which is a cmd line tool, but seems to already be getting the tree data somewhere -> perhaps there’s a way to do tree-vis without changes to H2O.

@micahstubbs
Copy link
Contributor

comment from @ivy_wang ^^

@micahstubbs
Copy link
Contributor

an idea: use the mojo tree viewer @tomkraljevic wrote to QA this.
or we could just use that graph builder directly.

per @tomkraljevic

there were some tricky things to figure out regarding categorical level and N/A reachability calculations as each level is traversed, and rows are eliminated level-by-level.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants