-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Reactivity gets broken #4627
Comments
Hey @XantreGodlike, thanks for reporting this issue. Can you please check if PR #4633 fixes it? |
I will try tomorrow |
I've created new branch with applied patch, and this bug still reproducing |
I think its unrelated with useAnimatedReaction, it about reactivity system, useDerivedValue stopping to execute too. Values changes, but dependecies is not updating |
@tomekzaw what do you think about it? |
@XantreGodlike I haven't had time to investigate this issue yet. |
Reactivity in 2.17.0 seems to be work fine |
2.17.0: ok https://github.com/XantreGodlike/reanimated-broken-reactivity/tree/reanimated2-check I think it should be helpful |
It's critical bug, and blocks our bug fix, because we are using |
@tomekzaw do you want I try to implement actual runtime reactivity system instead, this strange one. I think preact signals would look great for this purpose |
Any fixes found for this issue? I'm experiencing it on a simple |
Which version of reanimated do you use? Seems to be it was fixed in 3.4 |
Closing since the problem is allegedly fixed in 3.4 |
Description
Reactivity gets broken after many sharedValue assignments.
How its gets broken in reality
Steps to reproduce
useDerivedValue
oruseAnimatedReaction
Snack or a link to a repository
https://github.com/XantreGodlike/reanimated-broken-reactivity
Reanimated version
3.3.0
React Native version
0.71.7
Platforms
Android
JavaScript runtime
Hermes
Workflow
Expo bare workflow
Architecture
Paper (Old Architecture)
Build type
None
Device
Android emulator
Device model
No response
Acknowledgements
Yes
The text was updated successfully, but these errors were encountered: