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

Make demo display corresponding commandline syntax for current sequence #140

Closed
jywarren opened this issue Oct 14, 2017 · 5 comments
Closed

Comments

@jywarren
Copy link
Member

This would be great to emphasize in the demo how the same command can be run from the CLI interface.

@jywarren
Copy link
Member Author

Here, so what we could do is to make a function that could try to read through all the steps and settings, and assemble a CLI string from them. The CLI syntax is here:

https://github.com/publiclab/image-sequencer#cli-usage

The CLI also can take multiple steps at once, like so:

    $ ./index.js -i [PATH] -s "step-name-1 step-name-2 ..."

But for this, double quotes must wrap the space-separated steps.

Options for the steps can be passed in one line as json in the details option like

$ ./index.js -i [PATH] -s "brightness" -d '{"brightness":50}'

But i'm not sure if it can be chained with details too --

@marzann -- i'm not 100% sure this is going to be easy before there is some work done to the CLI code, here:

https://github.com/publiclab/image-sequencer/blob/master/index.js

In the meantime, would you like to try something from another project in our community? Maybe this one?

publiclab/plots2#2493

@marzann
Copy link

marzann commented May 1, 2018

@jywarren thank you, I will look over your notes on this!

@jywarren jywarren added this to the Default UI work milestone May 5, 2018
@jywarren
Copy link
Member Author

Hi, @marzann -- we've gone ahead and merged in the new demo code - in #214 -- a big change! Demo code is now in /examples/lib/____.js. Maybe one of these would be interesting to try?

#232

Take a look!

@tech4GT
Copy link
Member

tech4GT commented Jul 16, 2018

@jywarren taking this up now!!

@jywarren
Copy link
Member Author

jywarren commented Jul 17, 2018 via email

tech4GT added a commit to tech4GT/image-sequencer that referenced this issue Jul 21, 2018
@ghost ghost assigned tech4GT Jul 21, 2018
@ghost ghost added the in-progress label Jul 21, 2018
@ghost ghost removed the in-progress label Aug 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants