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

Layer type Eltwise is not implemented #7

Open
duanhai opened this issue Mar 13, 2018 · 2 comments
Open

Layer type Eltwise is not implemented #7

duanhai opened this issue Mar 13, 2018 · 2 comments
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@duanhai
Copy link

duanhai commented Mar 13, 2018

Hi,there.Could you add the implementation of ELtwise layer in your repo.Thx.

@chaosmail chaosmail added the enhancement New feature or request label Mar 13, 2018
@chaosmail
Copy link
Owner

Hi @duanhai

thanks for the suggestion. Currently I am waiting for the finalized specs of the deeplearnjs-tensorflow integration (https://github.com/PAIR-code/deeplearnjs/issues/637) to update my lib accordingly. I am using an older API version of deeplearnjs as well.

However, implementing this layer should be quite straightforward, you could try to add the layer as well. See https://github.com/chaosmail/deeplearnjs-caffe/blob/master/src/layer.ts#L60 for the other ops and let me know if you need some more help!

Best, Christoph

@duanhai
Copy link
Author

duanhai commented Mar 14, 2018 via email

@chaosmail chaosmail added the good first issue Good for newcomers label Mar 20, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants