-
Notifications
You must be signed in to change notification settings - Fork 92
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
Stop viewabilityTimer when viewForTracking has become nil #908
Comments
Hi @shinwan2 ! Thanks for the good finding! I wouldn't expose However, it totally makes sense to review CPU consumption, especially after the ad is already displayed and all needed trackers are processed. Can you share the part of your logs related to the Prebid SDK so we can study it to address the issue? |
Hi @YuriyVelichkoPI. checkViewability logs
I understand your concern. After https://github.com/prebid/prebid-mobile-ios/pull/900/files#diff-a11c923043093ee266f7b9a42a9ca8e6e508fc0e7078815a469a10e635f1b62bR39 I saw that the |
Thanks for the logs and the advice, @shinwan2 ! @alexsavelyev, this ticket should be prioritized for the next release. |
Is your feature request related to a problem? Please describe.
I'm trying to place an ad as a
UITableViewCell
for every 4 contents. It works great by usingbut when I scroll down, the log becomes too noisy because invisible ads keep their viewability timer running.
I feel it's not CPU-friendly when I have more than 10 ads in a list.
Describe the solution you'd like
Can we expose
unregisterViewFromTracking
inNativeAd
so I can call it like below?Describe alternatives you've considered
I've tried using
but the timer keeps running.
The text was updated successfully, but these errors were encountered: