-
Notifications
You must be signed in to change notification settings - Fork 65
Add L2/L8 plots and extend summary info/plots with L5 offsets, L8 trims, and L9 MDP #353
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
Conversation
@bbeny123 thanks again for also adding plot for L8 metadata. One last request: I also noticed that in the examples you show for Summary ( |
@manuelrn As for the |
It seems fine to me however L9 is also dynamic. |
Eventually I think it will be worthwhile to have an extra CLI config to limit the trim fields because the current L8 trims are way too crowded. |
Changes:
|
Couldn't manage to push to the branch so merge it is. |
This MR introduces two functional updates:
1.
--plot-type
option forplot
Adds support for plotting based on
L2
andL8
metadata.2. Extended metadata info in summary and plots
Additional metadata fields are now included in both the
info -s
andplot
annotations:CM v4.0
RPUs only)CM v4.0
RPUs only)Examples:
1.
info -s
Current CM v2.9:
Current CM v4.0:
New CM v2.9 + L5 0:
New CM v2.9 + L5 Standard:
New CM v2.9 + L5 Variable Top/Bottom:
New CM v4.0 + L5 Null:
New CM v4.0 + L5 Variable:
2.
plot
Current CM v2.9:

Current CM v4.0:

New CM v2.9 + L5 0:

New CM v2.9 + L5 Variable Top/Bottom:

New CM v4.0 + L5 Null:

2.
plot --plot-type l2
New CM v2.9 + L5 Standard (flat chart, no manual trims):

New CM v4.0 + L5 Variable:

New CM v4.0 + L5 Standard:

3.
plot --plot-type l8t
4.
plot --plot-type l8s
5.
plot --plot-type l8h