-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
React native upgrade 0.62.2 hermes rebased #1587
React native upgrade 0.62.2 hermes rebased #1587
Conversation
…upgrade-0.62.1-hermes-rebased
…upgrade-0.62.1-hermes-rebased
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🤞
…upgrade-0.62.1-hermes-rebased
…upgrade-0.62.1-hermes-rebased
@andrepimenta, @estebanmino, @omnat It is in my opinion that v8 is still smoother than hermes; see performance comparisons below: In all screen casts, this branch in release mode is on left (Samsung Note 9 device) and playstore is on right (Google Pixel 3 XL) side by side comparison of devices if interested = https://www.gsmarena.com/compare.php3?idPhone1=9163&idPhone2=9257 I am connected to the same wifi on each device Launch app > drawer > wallet Notice how long it takes the drawer to open and how long it takes to go to wallet. I've noticed that v8 responds faster to touch in comparison to hermes seen here = https://recordit.co/IxFT5Tyev1 Switching networks on wallet view > tapping on collectibles seen here = https://recordit.co/VdW77s4P9D switching accounts on collectibles view > going to tokens > open drawer > switch accounts > go to collectibles > go to txn history and scroll > tap on a txn > view on etherscan > go to wallet > tokens > search token > tap token > select token > remove token seen here = https://recordit.co/rFEbuacIa2 on wallet view, switch networks > tap on ETH asset > tap send > transfer between accounts > select account > enter amount > on confirm tap edit and edit gas > send txn The app actually crashed on hermes branch in this example seen here = https://recordit.co/s7GdUuof81 on browser, go to faucet > donate > connect > Edit > change gas and amount > send > view on etherscan this one was pretty much the same in regards to performance seen here = https://recordit.co/9JpQMNlSug Send collectible flow on mainnet (there was a crash on hermes branch) seen here = https://recordit.co/90csr3fE96 These are just some examples. Overall I feel like v8 definitely responds quicker to touch and just feels smoother overall IMO Feel free to take a look on your devices to see if it's the same experience for you. |
…upgrade-0.62.1-hermes-rebased
I guess Hermes is still too new to be added now. I also found this performance comparison: So I think it's better for now to stick with our V8. I also updated our V8 to the newest version (on the updated libs branch). Will close this for now and we can come back to Hermes later. |
This is the react native 0.62.2 upgrade with the Hermes engine enabled. The code that enables Hermes can be found here.
The Hermes documentation can be found here.
The performance tests should be done in release mode.
For other info about the upgrade check #1586.
Issue
Resolves #1201 (Hermes point)