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

After replacing the version, the cpu usage soared, #35

Open
wisepmlin opened this issue May 3, 2022 · 14 comments
Open

After replacing the version, the cpu usage soared, #35

wisepmlin opened this issue May 3, 2022 · 14 comments
Labels
need more details Try making a minimal reproducible example

Comments

@wisepmlin
Copy link

上传截屏2022-05-04 00.13.12.png...
The source of the problem to be detected

@wisepmlin
Copy link
Author

使用最新1.3.2版本

@wisepmlin
Copy link
Author

Bound preference ElementPreferenceKey tried to update multiple times per frame.
Bound preference ContentFrameReaderPreferenceKey tried to update multiple times per frame.
Bound preference ContentFrameReaderPreferenceKey tried to update multiple times per frame.
Bound preference ContentFrameReaderPreferenceKey tried to update multiple times per frame.
Bound preference ContentFrameReaderPreferenceKey tried to update multiple times per frame.
Bound preference ContentFrameReaderPreferenceKey tried to update multiple times per frame.
Bound preference ContentFrameReaderPreferenceKey tried to update multiple times per frame.
Bound preference ContentFrameReaderPreferenceKey tried to update multiple times per frame.
Bound preference ContentFrameReaderPreferenceKey tried to update multiple times per frame.
Bound preference ContentFrameReaderPreferenceKey tried to update multiple times per frame.
Bound preference ContentFrameReaderPreferenceKey tried to update multiple times per frame.
Bound preference ContentFrameReaderPreferenceKey tried to update multiple times per frame.
Bound preference ContentFrameReaderPreferenceKey tried to update multiple times per frame.

@aheze aheze added the need more details Try making a minimal reproducible example label May 3, 2022
@aheze
Copy link
Owner

aheze commented May 3, 2022

I can't open the png, could you upload it again?

@wisepmlin
Copy link
Author

截屏2022-05-04 02 06 26

@wisepmlin
Copy link
Author

After closing the popover, the window still occupies the cpu

@wisepmlin
Copy link
Author

My interaction is in the page, some of them occupy more than 90% of the CPU, and the page is seriously stuck.

@aheze
Copy link
Owner

aheze commented May 4, 2022

Huh. Is your app running on iOS? Try making a minimal reproducible example so I can reproduce

@wisepmlin
Copy link
Author

RPReplay_Final1651814987.MP4

@wisepmlin
Copy link
Author

截屏2022-05-06 13 31 25

@wisepmlin
Copy link
Author

截屏2022-05-06 13 30 57

@wisepmlin
Copy link
Author

截屏2022-05-06 13 30 36

@aheze
Copy link
Owner

aheze commented May 6, 2022

That looks really weird. So it was working fine back in v1.3.1, but broke in v1.3.2? I didn't change the code much. The problem is probably here though: .onValueChange(of: transaction?.animation) { _, _ in.

I'll take a look next week (got a bunch of exams right now...) but feel free to investigate or make a PR yourself if you want.

@wisepmlin
Copy link
Author

Development is tight, I first return to the system components, there is time to study the reasons

@wisepmlin
Copy link
Author

The height of the encapsulated component with uikit's UItextKit creates a refresh conflict in LazyVStask's container, causing rendering to bounce

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
need more details Try making a minimal reproducible example
Projects
None yet
Development

No branches or pull requests

2 participants