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

fix direct_returnn_layer_call() for multiple bases #131

Open

vieting wants to merge 2 commits into main from peter_bases

Contributor

vieting commented Dec 7, 2022

We introduced the direct_returnn_layer_call() method in #129. I extended it a bit so it can handle modules with multiple bases.


          fix direct_returnn_layer_call() for multiple bases

5bd5fa1

vieting requested a review from albertz

December 7, 2022 10:13

Contributor Author

vieting commented Dec 7, 2022

The failing tests are the ones from #125.

albertz requested changes

View reviewed changes

pytorch_to_returnn/torch/nn/modules/module.py Outdated Show resolved Hide resolved


          queue instead of recursion

89ef0e8

vieting requested a review from albertz

December 7, 2022 15:51

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

Labels

None yet