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

regularization and constraints for Convolution1D and Convolution2D #218

Merged
merged 2 commits into from
Jun 12, 2015

Conversation

phreeza
Copy link
Contributor

@phreeza phreeza commented Jun 12, 2015

fix for #170, including constraints. Nothing fancy, just copied the code from the dense layer, but I suppose it should work the same.

@phreeza phreeza changed the title Straightforward copying of regularization and constraints regularization and constraints for Convolution1D Jun 12, 2015
@phreeza phreeza changed the title regularization and constraints for Convolution1D regularization and constraints for Convolution1D and Convolution2D Jun 12, 2015
@fchollet
Copy link
Member

Yes, this should be enough. Did you test it, and did you observe better results (reduced overfitting) on some test task?

@phreeza
Copy link
Contributor Author

phreeza commented Jun 12, 2015

Yes, seems to work:

https://gist.github.com/phreeza/ba64c8fd26d331576fe2

@fchollet
Copy link
Member

Great : )

fchollet added a commit that referenced this pull request Jun 12, 2015
Add regularization and constraints for Convolution1D and Convolution2D
@fchollet fchollet merged commit 161d47b into keras-team:master Jun 12, 2015
fchollet pushed a commit that referenced this pull request Sep 22, 2023
* Add scatter_update and block_update

* Add docstring

* remove explicit conversion

* Fix docstring
hubingallin pushed a commit to hubingallin/keras that referenced this pull request Sep 22, 2023
* Add scatter_update and block_update

* Add docstring

* remove explicit conversion

* Fix docstring
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