Skip to content

Extend SliceNdLayer to use a layer as the size argument#670

Merged
albertz merged 5 commits into
rwth-i6:masterfrom
robin-p-schmitt:schmitt-slicend-size-as-layer
Sep 20, 2021
Merged

Extend SliceNdLayer to use a layer as the size argument#670
albertz merged 5 commits into
rwth-i6:masterfrom
robin-p-schmitt:schmitt-slicend-size-as-layer

Conversation

@robin-p-schmitt

Copy link
Copy Markdown
Contributor

Right now, start and size need to have the same shape when they are given as layers. This suffices for my use cases but we can also add support for different shapes if you think we need them.

Comment thread returnn/tf/layers/basic.py Outdated
Comment thread returnn/tf/layers/basic.py
Comment thread returnn/tf/layers/basic.py Outdated
Comment thread returnn/tf/layers/basic.py
Comment thread returnn/tf/layers/basic.py Outdated
@albertz

This comment has been minimized.

Comment thread returnn/tf/layers/basic.py
Comment thread returnn/tf/layers/basic.py Outdated
Comment thread returnn/tf/layers/basic.py
Comment thread returnn/tf/layers/basic.py Outdated
Comment thread returnn/tf/layers/basic.py
Comment thread returnn/tf/layers/basic.py Outdated
@albertz

This comment has been minimized.

@albertz albertz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good now

@albertz albertz merged commit 3f1554d into rwth-i6:master Sep 20, 2021
@albertz

albertz commented Sep 21, 2021

Copy link
Copy Markdown
Member

Just for reference: Related was #661 and #635

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