-
Notifications
You must be signed in to change notification settings - Fork 35
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
Feature: FPP Visualize #147
Conversation
Fails tests because |
Ready for review, as all the pieces are there, see nasa/fprime#2059 (comment) |
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.
A few questions, but looks generally good.
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 had one more minor question, but not sure it should trip up the PR.
Change Description
Depends on:
Introducing:
fprime-util visualize
Within a Topology directory, performs the following
Effectively, this allows with a single command to spin up a visualization of the topology and all its subtopology graphs.
cd Ref/Top fprime-util visualize
Rationale
Major QoL improvement to visualization capabilities. This is currently doable, but requires a lot of manual installation of tools etc...
Todo
Note
Opening as draft in case there's change requests. Do not merge for now, there are a couple of dependencies for this that need to be released before this can go through.