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
ortbConverter: send max tmax #10964
Comments
The actual timeout is higher than the ceiling, why not just ask the publisher to set an actual ceiling they are comfortable with? |
Are you suggesting that pubs just set s2sConfig.timeout to be the same as what's sent in pbjs.requestBids() and use that as the ceiling? I would solidly prefer not to have to (A) document that and (B) get existing pubs to change the page if/when they want to try this feature when it's easy and effective to get the client-side timeout. |
from here:
AFAIK this has never been the case since at least my time with Prebid. Do we want to also implement something like it - maybe a fixed 500ms discount? (percentage doesn't make a lot of sense to me, if I increase client-side timeout it's not like the latency to server also increases). |
Hmm. Good topic. I think 500ms would be an overly large buffer. The number depends a lot on the user's network proximity to the nearest Prebid Server. Discuss in a future meeting? |
Discussed. Agreed that a publisher should be able to define the tmaxmax like this:
The ortbConverter library should check for the existence of |
@bretg I can also add that the below does not seem to be the working
Tried setting timeout in s2s to 9000 and timeout in requestbids is 600, but tmax is still 9000 in the request. |
as discussed in prebid/Prebid.js#10964
Thanks for confirming @bmjoju - fixed the docs page. |
@bretg can you remind me, is Magnite picking this up? |
Should we track this as a different bug? |
Code for PBS-Java looking at tmaxmax is in QA.
Not sure how that could be. @bmjoju - what version of PBJS are you running. The latest code in the ortbconverter does this:
|
@bretg It is version v8.31.0. It was this in the documentation that didnt work and you also fixed the docs:
The tmax is working and set to what im setting timeout to in the s2sConfig. But the docs just said that the timeout in the s2sConfig config would be adjusted if set too high than the prebid.js timeout used in requestbids. |
Type of issue
enhancement
Description
The ortbConverter lib sets the ORTB tmax value to s2sConfig.timeout. This is great, but we have a module that would like to know the actual client-side timeout -- the
timeout
passed to pbjs.requestBids.Specifically the use case is timeout optimization. We're exploring a server-side module that would help the publisher optimize the timeout value, essentially ignoring
tmax
and experimenting with other, possibly more effective values. But we need to know the ceiling for how far these explorations should go.We propose passing the client-side timeout value as
ext.tmaxmax
.The text was updated successfully, but these errors were encountered: