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

Add gamma module #374

Merged
merged 2 commits into from
Oct 3, 2018
Merged

Add gamma module #374

merged 2 commits into from
Oct 3, 2018

Conversation

Mridul97
Copy link

@Mridul97 Mridul97 commented Oct 2, 2018

Issue #368

Changes : A module is added which applies gamma correction on an image and takes gamma as an input.

@Mridul97
Copy link
Author

Mridul97 commented Oct 2, 2018

This is a screenshot of the output with gamma = 0.2 :

ss_13

@jywarren
Copy link
Member

jywarren commented Oct 2, 2018

Wow, very cool! Maybe add a link to Wikipedia's page on Gamma to the description? @tech4GT what do you think of this? 👍

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.

This looks very nice!! Awesome work @Mridul97
I was thinking let's call this module Gamma Correction, seems more descriptive to me, what say?

"inputs": {
"adjustment": {
"type": "float",
"desc": "gamma correction for the new image",
Copy link
Member

Choose a reason for hiding this comment

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

Just to be extra clear let's also mention this is inverse of the actual gamma.

Copy link
Author

@Mridul97 Mridul97 Oct 3, 2018

Choose a reason for hiding this comment

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

I will change the module name to Gamma Correction and also add a link to Wikipedia's page on Gamma to the description. Thanks!

I didn't quite get 'inverse of the actual gamma'. I think it is gamma, how is this inverse of the actual gamma? Sorry.

Copy link
Member

Choose a reason for hiding this comment

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

Oh Actually the gamma correction is the inverse of the actual gamma factor, so just to be extra clear I was thinking maybe we should mention that this is the gamma correction, inverse of the actual gamma factor. Thanks!

Copy link
Author

@Mridul97 Mridul97 Oct 3, 2018

Choose a reason for hiding this comment

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

Oh okay, I will do that! Thanks!

@Mridul97
Copy link
Author

Mridul97 commented Oct 3, 2018

I have made the changes. Please have a look!

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.

Very nicely done!! I think we are ready to merge here!!🎉

@jywarren
Copy link
Member

jywarren commented Oct 3, 2018

Great :-) 👍 👍 👍

Now that @Mridul97 knows the module format, they can help other people get their modules into the right format! One thing that can help is to have a "Saved reply" which links to the module docs.

https://github.com/settings/replies

It could have a friendly message and link to: https://github.com/publiclab/image-sequencer/blob/master/CONTRIBUTING.md

Actually we could also have a pull request template linking to that as well! See some of the templates we can install here:

https://github.com/publiclab/plots2/tree/master/.github/

@jywarren jywarren merged commit 338e610 into publiclab:main Oct 3, 2018
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 <asingla590@gmail.com>

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

* gif button name changed

* fixes #383 (#396)

Signed-off-by: Varun Gupta <varun.gupta1798@gmail.com>

* fix (#387)

Signed-off-by: Varun Gupta <varun.gupta1798@gmail.com>

* 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 <asingla590@gmail.com>

* 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 <varun.gupta1798@gmail.com>

* add option to drop gradient

Signed-off-by: Varun Gupta <varun.gupta1798@gmail.com>

* fix bug

Signed-off-by: Varun Gupta <varun.gupta1798@gmail.com>
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