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

Implement client ads pacing mechanism #11057

Closed
AndriusA opened this issue Aug 3, 2020 · 1 comment · Fixed by brave/brave-core#6295
Closed

Implement client ads pacing mechanism #11057

AndriusA opened this issue Aug 3, 2020 · 1 comment · Fixed by brave/brave-core#6295

Comments

@AndriusA
Copy link

AndriusA commented Aug 3, 2020

Description

Implement client-side support for pass-through rate ads pacing. Rationale and calculations described in detail:

Steps to Reproduce

Ads, especially with broad targeting criteria, currently get shown at the highest rate close to midnight when daily budgets reset. Better pacing throughout the day should yield better results for both advertisers and users.

Actual result:

Ads often spending bulk of the daily budget within ~15-17 hours

Expected result:

Ads should spend their daily budget in closer to the full 24 hours, while still spending the full daily budget. More budget (and impressions) should be allocated to times of day when there are more active users.

Reproduces how often:

Easily calculated from data

Brave version (brave://version info)

All versions

Version/Channel Information:

  • Can you reproduce this issue with the current release?
  • Can you reproduce this issue with the beta channel?
  • Can you reproduce this issue with the nightly channel?

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields?
  • Does the issue resolve itself when disabling Brave Rewards?
  • Is the issue reproducible on the latest version of Chrome?

Miscellaneous Information:

@AndriusA AndriusA self-assigned this Aug 3, 2020
@tmancey tmancey added enhancement priority/P3 The next thing for us to work on. It'll ride the trains. QA/Yes release-notes/include labels Aug 5, 2020
@tmancey tmancey changed the title [Desktop] Implement client ads pacing mechanism Implement client ads pacing mechanism Aug 5, 2020
@tmancey tmancey added the OS/Android Fixes related to Android browser functionality label Aug 5, 2020
@tmancey tmancey added this to the 1.14.x - Nightly milestone Aug 11, 2020
@rebron rebron added QA/No release-notes/exclude and removed QA/Yes priority/P3 The next thing for us to work on. It'll ride the trains. release-notes/include labels Sep 9, 2020
@btlechowski
Copy link

The pacing does not work at all. Logged #11643.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants