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.

Sign up for GitHub

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

Jump to bottom

Merge layer should support arbitrary number of inputs #250

Open
bauerfe opened this issue Sep 18, 2024 · 2 comments
Open

Merge layer should support arbitrary number of inputs #250

bauerfe opened this issue Sep 18, 2024 · 2 comments
Assignees
@bauerfe
Labels
doing Work in Progress enhancement New feature or request

Comments

@bauerfe
Copy link
Contributor

bauerfe commented Sep 18, 2024

Currently the Merge layer merges exactly two inputs. However, there is no restriction on DynapCNN or Speck that would limit the number of inputs to a layer to 2. I suggest to make Merge more flexible.
@bauerfe bauerfe added enhancement New feature or request doing Work in Progress labels Sep 18, 2024
@bauerfe bauerfe self-assigned this Sep 18, 2024
@bauerfe
Copy link
Contributor Author

bauerfe commented Sep 18, 2024

Note that this branch addresses the issue.

@bauerfe
Copy link
Contributor Author

bauerfe commented Nov 14, 2024

Still to do in branch mentioned above:

  • Unit test
  • Update docs
  • Update Changelog

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bauerfe bauerfe

Labels
doing Work in Progress enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

When branches are created from issues, their pull requests are automatically linked.

dev/merge_multiple_inputs synsense/sinabs
1 participant
@bauerfe