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

fix: init hidden state for reactant #1026

Merged
merged 7 commits into from
Nov 15, 2024
Merged

fix: init hidden state for reactant #1026

merged 7 commits into from
Nov 15, 2024

Conversation

avik-pal
Copy link
Member

@avik-pal avik-pal commented Nov 4, 2024

fixes #1025

@avik-pal
Copy link
Member Author

Once we tag a new Reactant release I can add tests and merge this

@avik-pal avik-pal force-pushed the ap/rnn_reactant branch 2 times, most recently from aec71c6 to c0e4564 Compare November 11, 2024 21:31
Copy link
Contributor

github-actions bot commented Nov 12, 2024

Benchmark Results (ASV)

main 7f32c98... main/7f32c985c71041...
basics/overhead 0.127 ± 0.0015 μs 0.138 ± 0.0012 μs 0.921
time_to_load 1.18 ± 0.0065 s 1.2 ± 0.0022 s 0.987

Benchmark Plots

A plot of the benchmark results have been uploaded as an artifact to the workflow run for this PR.
Go to "Actions"->"Benchmark a pull request"->[the most recent run]->"Artifacts" (at the bottom).

@avik-pal
Copy link
Member Author

only waiting for a new Reactant release. Then I will bump the minimum version requirements

@avik-pal avik-pal marked this pull request as ready for review November 14, 2024 18:06
@avik-pal
Copy link
Member Author

need to fix some of the scalar indexing issues now that scalar indexing is disabled in Reactant for non-CPU buffers

@avik-pal avik-pal merged commit bbf5033 into main Nov 15, 2024
59 of 75 checks passed
@avik-pal avik-pal deleted the ap/rnn_reactant branch November 15, 2024 01:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Compiling Recurrent Models with Reactant
1 participant