apache / incubator-weex

Apache Weex (Incubating)
https://weex.apache.org
Apache License 2.0
13.74k stars 1.81k forks source link

[Android]Weex pages in 0.26 sdk version are rendering slower than 0.24 version #2718

Open nikkulshrestha opened 5 years ago

nikkulshrestha commented 5 years ago

Describe the bug

While upgrading to weex SDK version 0.26, pages rendering speed has become very slow. This is also impacting events also. In 0.24 version everything was working fine.

To Reproduce

Just load weex pages with some events and check behaviour.

Environment

Weex SDK: 0.26 Platform: Android

Expected behavior

Pages rendering and event speed should be as fast as render in SDK version 0.24.

lucky-chen commented 5 years ago

@nikkulshrestha Can you provide demo case? such as link ,so i can reproduce and debug

lucky-chen commented 5 years ago

if you are in debug mode , then will print much log , try close log ,and test

nikkulshrestha commented 5 years ago

Thanks @lucky-chen, we will check release build also.

lucky-chen commented 5 years ago

@nikkulshrestha provide a demo, so i can debug and found the problem

lucky-chen commented 5 years ago

@nikkulshrestha you can change branch with master ,and build new sdk with this

-./gradlew weex_sdk:clean weex_sdk:assembleRelease PbuildRuntimeApi=false  --info

or change build.gradle

 def buildRuntimeApi = project.hasProperty('buildRuntimeApi') ? project.property('buildRuntimeApi') : false

then test

lucky-chen commented 5 years ago

find the problem https://github.com/apache/incubator-weex/pull/2736 ,and will fix in next release verison @nikkulshrestha

vipulgupta29 commented 5 years ago

@lucky-chen - Please share the ETA for next release.

vipulgupta29 commented 5 years ago

Hi @lucky-chen - For now we are using 0.26.1.3-SNAPSHOT sdk version for our project. It is working fine on 64 bit devices. But we have experienced the 10-15% increase in rendering time on 32 bit devices. Request your support in this context.

lucky-chen commented 5 years ago

@vipulgupta29 ok,i will check with this demo : http://dotwe.org/vue/3c1dca8c042ebffb32d2b7be86420012

vipulgupta29 commented 5 years ago

In the apk that we have build, there are 3 folders arm64-v8a, armeabi-v7a and x86, but no x86_64

lucky-chen commented 5 years ago

In the apk that we have build, there are 3 folders arm64-v8a, armeabi-v7a and x86, but no x86_64

please create new issue for x86_64 , we will distribution other people to look this issue。

YorkShen commented 5 years ago

After switching to jsc-android, Weex is capable of supporting x86-64.

@vipulgupta29 Could you please fix this problem by yourself and create a PR ?

I have a busy daily schedulue, it's unlikely for myself to fix it in recent month`