-
Notifications
You must be signed in to change notification settings - Fork 32
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
Sweep Visualizations #245
Sweep Visualizations #245
Conversation
[pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci
2c9007c
to
181283f
Compare
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.
Mostly small things to change
…to visualizations
for more information, see https://pre-commit.ci
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.
Please add the commands also to the readme with examples:
Current usage
elk plot plots your most recent sweep
elk plot --sweep [sweep_name] to visualize particular sweep
elk sweep ... --visualize visualizes after sweeping
Future PRs
elk plot --sweep [sweep_1 .. sweep_n] plots other sweeps
data validation per plot
tests
elk plot --elicit
@lauritowal any other comments? |
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.
There are still some small things, which I would personally change, but I guess it's okay for now
Current usage
elk plot
plots your most recent sweepelk plot --sweep [sweep_name]
to visualize particular sweepelk sweep ... --visualize
visualizes after sweepingFuture PRs
elk plot --sweep [sweep_1 .. sweep_n]
plots other sweepsdata validation per plot
tests
elk plot --elicit