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 sine activation #138

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

dcato98
Copy link

@dcato98 dcato98 commented Aug 6, 2020

Dear Playground Maintainers,

I would like to propose adding the sine activation to the playground.

This addition has significantly helped me develop an intuition behind the recently published "SIREN" paper (Implicit Neural Representations with Periodic Activation Functions by Vincent Sitzmann, Julien N. P. Martel, et al.), despite my lack of formal training in signal processing and related fields. For example, playing with this feature makes it apparent how NNs using the sine activation can use fewer neurons to represent transformations from coordinates to pixels (compared to the other activation functions in the playground). It helps to be able to visualize the relationships between the frequency of the signals in relation to the weights, biases, and learning rate. I also now have a better understanding how several failure modes develop.

I can appreciate not wanting to overwhelm users with too many choices of activation function. I think an exception should be made in this case since the behavior of the sine activation is sufficiently different from the others and is increasingly relevant in neural representation research. But don't take my word for it, try it out for yourself in this fork!

I hope you will consider adding this feature!

All the best,
David

@Zi7ar21
Copy link

Zi7ar21 commented Mar 8, 2021

This is like the easiest commit in the world to just click merge on, what are the people managing this repository even thinking

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants