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

adds default values to input fields for all module steps ( fixes #382 ) #384

Merged
merged 3 commits into from
Oct 5, 2018

Conversation

Ankit-Singla
Copy link

Provides default input values in input of all step modules to indicate the input values on which the module step is running. This also provides a function signature for inputs that require a function.

Copy link
Member

@tech4GT tech4GT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good work @Ankit-Singla
I think we can merge here🎉

@tech4GT
Copy link
Member

tech4GT commented Oct 5, 2018

@jywarren let's merge this one in!

@jywarren jywarren merged commit b300dc2 into publiclab:main Oct 5, 2018
@jywarren
Copy link
Member

jywarren commented Oct 5, 2018

🎉 👍 👍 👍

@jywarren
Copy link
Member

jywarren commented Oct 5, 2018

publishing it too! we'll need a new version number pretty soon!

@jywarren
Copy link
Member

jywarren commented Oct 5, 2018

@Ankit-Singla this is great. Would you want to take a look at this one too? It is UI centric and I think would be really cool to see: #247

Just in case it catches your eye! 🙌

@jywarren
Copy link
Member

jywarren commented Oct 5, 2018

Ah, oops! The defaults are great, but they override the URL-specified values! Take a look:

https://publiclab.github.io/image-sequencer/examples/#steps=convolution{constantFactor:0.1111111111111111|kernelValues:0 -1 0 -1 5 -1 0 -1 0}

These are then overridden by the defaults :-(

Would you mind taking a look? unfortunately this is not live on the demo!

Thank you!

@tech4GT
Copy link
Member

tech4GT commented Oct 5, 2018

Aha! I’ll see this!! Silly me!!

@Ankit-Singla
Copy link
Author

Sorry, but I didn't quite understand the meaning of 'overriding URL-specified values'. @jywarren
Can you please help me understand it better? @tech4GT

@tech4GT
Copy link
Member

tech4GT commented Oct 5, 2018

@Ankit-Singla the thing is that the input values that are taken from the url are being overriden by the value parameter, I'll figure this out!! Thanks for all the help!!

jywarren pushed a commit that referenced this pull request Oct 10, 2018
* added gif feature

* increased frame duration n changed gif btn name

* removed inline css

* improved ui

* #363 Add placeholder option to input (#370)

* Add placeholder option to input , change type of brightness input to number

 Changes to be committed:
	modified:   examples/lib/defaultHtmlStepUi.js
	modified:   src/modules/Brightness/info.json

* change 0% to 0 in brightness placeholder

* Add gamma module (#374)

* Add gamma module

* update description

* Add matrix math module (#358)

* Add matrix math module

* add info.json file

* correct format of module

* Add a constant factor input field

* clone the pixels array

* change default values

* add extra information on convolution module (#381)

* adds default values to input fields for all module steps ( fixes #382 ) (#384)

* added default values to input fields for all module steps

* merged

Signed-off-by: Ankit Singla <[email protected]>

* Accept type images in input field #364 (#366)

* gif button name changed

* fixes #383 (#396)

Signed-off-by: Varun Gupta <[email protected]>

* fix (#387)

Signed-off-by: Varun Gupta <[email protected]>

* updated

* Issue #392 Fixes alignment of message in Add step box (#393)

*  Fixes alignment of message in Add step box

* fixes css info id

* updates class selector in demo

* Spacing changes

* correctly regulates the behaviour of all links (#397)

Signed-off-by: Ankit Singla <[email protected]>

* enables Save button for an action if any of its input fields gain focus (#394)

* Updated index.html (#379)

* add histogram module (#354)

* add histogram module

Signed-off-by: tech4GT <[email protected]>

* add option to drop gradient

Signed-off-by: Varun Gupta <[email protected]>

* fix bug

Signed-off-by: Varun Gupta <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants