Image-sequencer: get settings updates in demo to only run() from current index

[jywarren]

Using the new version of Run!

[jywarren]

Actually I realized the new Run is a major version update because we've broken the previous version of Run. Can you also bump a major version?

[gitmate[bot]]

GitMate.io thinks the contributor most likely able to help you is @ccpandhare.

Possibly related issues are https://github.com/publiclab/image-sequencer/issues/130 (get CLI to run synchronously for chainability), https://github.com/publiclab/image-sequencer/issues/261 (Set of smaller issues for Run() and AddStep/InsertStep), https://github.com/publiclab/image-sequencer/issues/91 (Demo where steps can be set via URL hash), and https://github.com/publiclab/image-sequencer/issues/140 (Make demo display corresponding commandline syntax for current sequence).

[gitmate[bot]]

GitMate.io thinks the contributor most likely able to help you is @ccpandhare.

Possibly related issues are https://github.com/publiclab/image-sequencer/issues/130 (get CLI to run synchronously for chainability), https://github.com/publiclab/image-sequencer/issues/261 (Set of smaller issues for Run() and AddStep/InsertStep), https://github.com/publiclab/image-sequencer/issues/91 (Demo where steps can be set via URL hash), and https://github.com/publiclab/image-sequencer/issues/140 (Make demo display corresponding commandline syntax for current sequence).

[tech4GT]

@jywarren I think I did this in it too!! I did update the demo to run from current index on save and addstep, run from 0 when page is visited!!

[tech4GT]

bumping major version though!!

[tech4GT]

@jywarren Also can we putout a tweet about the sequencer 2.0 this is so exciting!!!!!

[jywarren]

:-)))

On Fri, Jun 8, 2018, 2:19 PM Varun Gupta [e-mail] wrote:

@jywarren https://github.com/jywarren Also can we putout a tweet about
the sequencer 2.0 this is so exciting!!!!!

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/publiclab/image-sequencer/issues/286#issuecomment-395845761,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABfJzHME-4RRBmGuNnQ3-Kk4kyT1IPuks5t6sAHgaJpZM4Ugvd7
.

[tech4GT]

Awesome!! Closing this via #287

[tech4GT]

@jywarren what issues are you facing specifically??

[jywarren]

I added a set of 3 invert operations, and when i delete the middle one, the
last one doesn't re-run. I think we should be able to write a test for this
sort of thing... but for now, manually testing it showed some
inconsistencies like this.

On Fri, Jun 8, 2018 at 2:43 PM, Varun Gupta [e-mail]
wrote:

@jywarren https://github.com/jywarren what issues are you facing
specifically??

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/publiclab/image-sequencer/issues/286#issuecomment-395852694,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABfJ31oXpRBWtyl9Q6AFwg5OQIWCh9xks5t6sXOgaJpZM4Ugvd7
.

[tech4GT]

@jywarren hmm this wierd, I just tried it and It worked

[tech4GT]

@jywarren are we serving the right files??

[jywarren]

Try opening this locally and then deleting the invert step:
http://localhost:8000/examples/#steps=import-image(url:%2Fexamples%2Fimages%2Fmonarch.png),saturation(saturation:0.5),channel(channel:green),invert()

On Fri, Jun 8, 2018 at 2:49 PM, Varun Gupta [e-mail]
wrote:

@jywarren https://github.com/jywarren hmm this wierd, I just tried it
and It worked

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/publiclab/image-sequencer/issues/286#issuecomment-395854487,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABfJxJfkpzrqxl5v70VC6JEZsvxRllBks5t6sdGgaJpZM4Ugvd7
.

[jywarren]

ER wait! Deleting the "channel" step i mean! The Invert step is then
unaffected for me?

On Fri, Jun 8, 2018 at 2:56 PM, Jeffrey Warren [e-mail] wrote:

Try opening this locally and then deleting the invert step:
http://localhost:8000/examples/#steps=import-image(
url:%2Fexamples%2Fimages%2Fmonarch.png),saturation(
saturation:0.5),channel(channel:green),invert()

On Fri, Jun 8, 2018 at 2:49 PM, Varun Gupta [e-mail]
wrote:

@jywarren https://github.com/jywarren hmm this wierd, I just tried it
and It worked

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/publiclab/image-sequencer/issues/286#issuecomment-395854487,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABfJxJfkpzrqxl5v70VC6JEZsvxRllBks5t6sdGgaJpZM4Ugvd7
.

[tech4GT]

@jywarren looking into this now!!

[tech4GT]

Ah small error!! found it!!

[tech4GT]

@jywarren should we now publish to npm??

[jywarren]

I already did!!!

On Fri, Jun 8, 2018 at 3:48 PM, Varun Gupta [e-mail]
wrote:

@jywarren https://github.com/jywarren should we now publish to npm??

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/publiclab/image-sequencer/issues/286#issuecomment-395869874,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABfJ995sIPTirTFsYTswSfD0xWJhuMfks5t6tT5gaJpZM4Ugvd7
.