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

wrong highest bid returned #2685

Closed
renebaudisch opened this issue Jun 6, 2018 · 8 comments
Closed

wrong highest bid returned #2685

renebaudisch opened this issue Jun 6, 2018 · 8 comments
Assignees

Comments

@renebaudisch
Copy link
Contributor

Type of issue

Seems like a bug

Description

Comparing "getHighestCpmBids" to "getAdserverTargeting" brings different results.

Steps to reproduce

It is happening only a few times so I cannot reliable reproduce, but when visiting https://www.formel1.de/news/grand-prix-berichte/2018-06-06/rennvorschau-montreal-mercedes-spielverderber-hypersoft you could see this happens sometimes.

Test page

https://www.formel1.de/news/grand-prix-berichte/2018-06-06/rennvorschau-montreal-mercedes-spielverderber-hypersoft

Expected results

"getHighestCpmBids" may not have lower CPM then "getAdserverTargeting"

Actual results

"getHighestCpmBids" sometimes gives lower CPM then "getAdserverTargeting"

Platform details

doesn't matter

Other information

A screenshot showing the issue:
prebid_bug

@stale
Copy link

stale bot commented Jun 20, 2018

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Jun 20, 2018
@renebaudisch
Copy link
Contributor Author

Closing when nobody even took a look is a very bad user experience...
It's not my fault that jaiminpanchal27 did not answer after14 days...

@stale stale bot removed the stale label Jun 20, 2018
@jaiminpanchal27
Copy link
Collaborator

@renebaudisch Sorry for not looking at this one. This might be because of this #2676.
We have a fix for this coming today.

@renebaudisch
Copy link
Contributor Author

So it's related, OK, thx, could you connect the issues?

@stale
Copy link

stale bot commented Jul 4, 2018

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Jul 4, 2018
@renebaudisch
Copy link
Contributor Author

I will close it when I checked that 1.15 will solve this...

@stale stale bot removed the stale label Jul 4, 2018
@jaiminpanchal27
Copy link
Collaborator

@renebaudisch Can you check and update the issue ?

@renebaudisch
Copy link
Contributor Author

looks good

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

No branches or pull requests

2 participants