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

Adding return_state to recurrent layers #39

Merged
merged 9 commits into from
Jan 13, 2025
Merged

Adding return_state to recurrent layers #39

merged 9 commits into from
Jan 13, 2025

Conversation

MartinuzziFrancesco
Copy link
Owner

@MartinuzziFrancesco MartinuzziFrancesco commented Dec 25, 2024

I am overlooking the return section in the docstrings for the layers adapted so far. Need to address that

@MartinuzziFrancesco MartinuzziFrancesco linked an issue Dec 25, 2024 that may be closed by this pull request
@MartinuzziFrancesco
Copy link
Owner Author

This will eventually need Flux 0.16.1 to release since we need scan to return both the states and the last state

ping @CarloLucibello

@MartinuzziFrancesco MartinuzziFrancesco merged commit d8e92ea into main Jan 13, 2025
4 checks passed
@MartinuzziFrancesco MartinuzziFrancesco deleted the fm/rs branch January 13, 2025 18:16
@MartinuzziFrancesco MartinuzziFrancesco restored the fm/rs branch January 20, 2025 11:00
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.

Adding return_state option
1 participant