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

Brave Ads epsilon greedy bandit is initialized twice and log is missing segment name #30053

Closed
tmancey opened this issue Apr 28, 2023 · 1 comment · Fixed by brave/brave-core#18291
Assignees

Comments

@tmancey
Copy link
Contributor

tmancey commented Apr 28, 2023

Description

Steps to Reproduce

  1. Fresh profile
  2. Join Rewards

Actual result:

[47924:259:0427/174004.738505:VERBOSE2:epsilon_greedy_bandit_processor.cc(53)] Epsilon greedy bandit arm was added for  segment
...
[47924:259:0427/174004.748017:VERBOSE1:epsilon_greedy_bandit_processor.cc(82)] Successfully initialized epsilon greedy bandit arms
[47924:259:0427/174004.748328:VERBOSE3:epsilon_greedy_bandit_processor.cc(40)] Epsilon greedy bandit arm already exists for architecture segment
...
[47924:259:0427/174004.749547:VERBOSE1:epsilon_greedy_bandit_processor.cc(82)] Successfully initialized epsilon greedy bandit arms

Appear in the console logs

Expected result:

[47924:259:0427/174004.738505:VERBOSE2:epsilon_greedy_bandit_processor.cc(53)] Epsilon greedy bandit arm was added for  architecture segment
...
[47924:259:0427/174004.748017:VERBOSE1:epsilon_greedy_bandit_processor.cc(82)] Successfully initialized epsilon greedy bandit arms

Should appear in the console logs

and after restarting the browser

[47924:259:0427/174004.748328:VERBOSE3:epsilon_greedy_bandit_processor.cc(40)] Epsilon greedy bandit arm already exists for architecture segment
...
[47924:259:0427/174004.749547:VERBOSE1:epsilon_greedy_bandit_processor.cc(82)] Successfully initialized epsilon greedy bandit arms

Should appear in the console logs

Reproduces how often:

Easily reproduced

Brave version (brave://version info)

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:

@btlechowski
Copy link

Verification passed on

Brave 1.53.99 Chromium: 114.0.5735.133 (Official Build) beta (64-bit)
Revision fbfa2ce68d01b2201d8c667c2e73f648a61c4f4a-refs/branch-heads/5735@{#1270}
OS Ubuntu 18.04 LTS

Verified test plan from the description

Verified epsilon greedy bandit is initialized once
image

Verified segments are present

[9428:9428:0620/112111.322713:VERBOSE2:epsilon_greedy_bandit_processor.cc(47)] Epsilon greedy bandit arm was added for architecture segment
[9428:9428:0620/112111.322884:VERBOSE2:epsilon_greedy_bandit_processor.cc(47)] Epsilon greedy bandit arm was added for arts & entertainment segment
[9428:9428:0620/112111.322884:VERBOSE2:epsilon_greedy_bandit_processor.cc(47)] Epsilon greedy bandit arm was added for automotive segment
[9428:9428:0620/112111.322884:VERBOSE2:epsilon_greedy_bandit_processor.cc(47)] Epsilon greedy bandit arm was added for business segment
[9428:9428:0620/112111.322884:VERBOSE2:epsilon_greedy_bandit_processor.cc(47)] Epsilon greedy bandit arm was added for careers segment
[9428:9428:0620/112111.322884:VERBOSE2:epsilon_greedy_bandit_processor.cc(47)] Epsilon greedy bandit arm was added for cell phones segment
[9428:9428:0620/112111.322885:VERBOSE2:epsilon_greedy_bandit_processor.cc(47)] Epsilon greedy bandit arm was added for drugs segment
[9428:9428:0620/112111.322885:VERBOSE2:epsilon_greedy_bandit_processor.cc(47)] Epsilon greedy bandit arm was added for education segment
[9428:9428:0620/112111.322885:VERBOSE2:epsilon_greedy_bandit_processor.cc(47)] Epsilon greedy bandit arm was added for family & parenting segment
[9428:9428:0620/112111.330442:VERBOSE2:epsilon_greedy_bandit_processor.cc(47)] Epsilon greedy bandit arm was added for fashion segment
[9428:9428:0620/112111.330573:VERBOSE2:epsilon_greedy_bandit_processor.cc(47)] Epsilon greedy bandit arm was added for folklore segment
[9428:9428:0620/112111.330669:VERBOSE2:epsilon_greedy_bandit_processor.cc(47)] Epsilon greedy bandit arm was added for food & drink segment
[9428:9428:0620/112111.330751:VERBOSE2:epsilon_greedy_bandit_processor.cc(47)] Epsilon greedy bandit arm was added for health & fitness segment
[9428:9428:0620/112111.330853:VERBOSE2:epsilon_greedy_bandit_processor.cc(47)] Epsilon greedy bandit arm was added for history segment
[9428:9428:0620/112111.330920:VERBOSE2:epsilon_greedy_bandit_processor.cc(47)] Epsilon greedy bandit arm was added for hobbies & interests segment
[9428:9428:0620/112111.331013:VERBOSE2:epsilon_greedy_bandit_processor.cc(47)] Epsilon greedy bandit arm was added for home segment
[9428:9428:0620/112111.331077:VERBOSE2:epsilon_greedy_bandit_processor.cc(47)] Epsilon greedy bandit arm was added for law segment
[9428:9428:0620/112111.331170:VERBOSE2:epsilon_greedy_bandit_processor.cc(47)] Epsilon greedy bandit arm was added for military segment
[9428:9428:0620/112111.331234:VERBOSE2:epsilon_greedy_bandit_processor.cc(47)] Epsilon greedy bandit arm was added for personal finance segment
[9428:9428:0620/112111.331330:VERBOSE2:epsilon_greedy_bandit_processor.cc(47)] Epsilon greedy bandit arm was added for pets segment
[9428:9428:0620/112111.331395:VERBOSE2:epsilon_greedy_bandit_processor.cc(47)] Epsilon greedy bandit arm was added for politics segment
[9428:9428:0620/112111.331491:VERBOSE2:epsilon_greedy_bandit_processor.cc(47)] Epsilon greedy bandit arm was added for real estate segment
[9428:9428:0620/112111.331560:VERBOSE2:epsilon_greedy_bandit_processor.cc(47)] Epsilon greedy bandit arm was added for religion segment
[9428:9428:0620/112111.331656:VERBOSE2:epsilon_greedy_bandit_processor.cc(47)] Epsilon greedy bandit arm was added for science segment
[9428:9428:0620/112111.331726:VERBOSE2:epsilon_greedy_bandit_processor.cc(47)] Epsilon greedy bandit arm was added for society segment
[9428:9428:0620/112111.331821:VERBOSE2:epsilon_greedy_bandit_processor.cc(47)] Epsilon greedy bandit arm was added for sports segment
[9428:9428:0620/112111.331889:VERBOSE2:epsilon_greedy_bandit_processor.cc(47)] Epsilon greedy bandit arm was added for technology & computing segment
[9428:9428:0620/112111.331984:VERBOSE2:epsilon_greedy_bandit_processor.cc(47)] Epsilon greedy bandit arm was added for travel segment
[9428:9428:0620/112111.332054:VERBOSE2:epsilon_greedy_bandit_processor.cc(47)] Epsilon greedy bandit arm was added for weather segment
[9428:9428:0620/112111.332154:VERBOSE2:epsilon_greedy_bandit_processor.cc(47)] Epsilon greedy bandit arm was added for crypto segment
[9428:9428:0620/112111.333394:VERBOSE1:epsilon_greedy_bandit_processor.cc(76)] Successfully initialized epsilon greedy bandit arms

@tmancey tmancey added this to Ads Jun 10, 2024
@tmancey tmancey moved this to Done in Ads Jun 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
3 participants