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

React native upgrade 0.62.2 hermes rebased #1587

Conversation

andrepimenta
Copy link
Member

@andrepimenta andrepimenta commented May 18, 2020

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)

@andrepimenta andrepimenta changed the title React native upgrade 0.62.1 hermes React native upgrade 0.62.1 hermes rebased May 18, 2020
@andrepimenta andrepimenta added needs-qa Any New Features that needs a full manual QA prior to being added to a release. next up prioritized to be picked up next labels May 18, 2020
Copy link
Contributor

@estebanmino estebanmino left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🤞

@andrepimenta andrepimenta changed the title React native upgrade 0.62.1 hermes rebased React native upgrade 0.62.2 hermes rebased May 18, 2020
@omnat
Copy link
Contributor

omnat commented May 19, 2020

React Native upgrade #2

@ibrahimtaveras00
Copy link
Contributor

@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.

@ibrahimtaveras00
Copy link
Contributor

ibrahimtaveras00 commented May 22, 2020

@ibrahimtaveras00 ibrahimtaveras00 added QA'd but questions A QA run through has been done but you need clarification on minor issues you found and removed needs-qa Any New Features that needs a full manual QA prior to being added to a release. labels May 22, 2020
@andrepimenta
Copy link
Member Author

andrepimenta commented May 22, 2020

I guess Hermes is still too new to be added now. I also found this performance comparison:
https://docs.google.com/spreadsheets/d/1XB6fuk-NYZbCDikxQOAJemE-P8cfbsfdXivTeVwcwIk/edit#gid=469794458

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
next up prioritized to be picked up next QA'd but questions A QA run through has been done but you need clarification on minor issues you found
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants