Skip to content

2.6-202312 Public Comment Feedback #83

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

Closed
hillslatt opened this issue Nov 15, 2023 · 11 comments
Closed

2.6-202312 Public Comment Feedback #83

hillslatt opened this issue Nov 15, 2023 · 11 comments

Comments

@hillslatt
Copy link
Collaborator

hillslatt commented Nov 15, 2023

Please comment on this issue with feedback on updates to the OpenRTB updates in Public Comment

Interest Group Auction Signaling

@bmayd
Copy link

bmayd commented Nov 16, 2023

Attribute names related to Interest Group Auctions are prefixed with "ig", might be clearer and easier to recognize them if they were prefixed with "iga".

@FlorentDancy
Copy link

Agreed with @bmayd, I believe we should use:

  • igab instead of igb (since it corresponds to InterestGroupAuctionBuyer)
  • igas instead of igs (since it corresponds to InterestGroupAuctionSupport)

In the same vein, I believe we should use pbps instead of ps (since it corresponds to perBuyerPrioritySignals)

@dmdabbs
Copy link
Contributor

dmdabbs commented Nov 17, 2023

Consider omitting Auction from the object names, as it's redundant. Everything in ORTB, and particularly this subset, ultimately involves an auction.

@bmayd
Copy link

bmayd commented Nov 17, 2023

Consider omitting Auction from the object names, as it's redundant. Everything in ORTB, and particularly this subset, ultimately involves an auction.

Yes, I think that makes sense so long as we don't anticipate other sorts of Interest Group related naming that might make "ig" too terse.

@aprokofg
Copy link

Regarding Interest Group Auction Signaling. What about perBuyerExperimentGroupIds? Should there be a field as part of the InterestGroupAuctionBuyer that allows to propagate experimentGroupId? Maybe something like eid field.

@dmdabbs
Copy link
Contributor

dmdabbs commented Nov 21, 2023

Hello. perBuyerExperimentIds were discussed, but omitted from this initial, provisional spec. "pending conversation around async calls with GAM, Prebid and SSPs."

See comments history in the working draft.

cc: @hillslatt

@liamwhiteside
Copy link

Should poddedupe also apply to Object: Audio ?
We send bid requests to DSPs asking for multiple imp objects to go into a single ad pod in an audio stream and will de-duplicate any responses that are for the same creative or advertiser domain etc. I'm not sure if any of the DSPs currently responding with duplicates would appreciate the extra signal and do anything with that information.

@piwanczak
Copy link

Wrt Interest Group Auction Signaling - shouldn't "biddable" default to 1?
Today, should a buyer receive for example a Native/Video/Display request or some other - it is implied that the buyer is allowed to participate/bid. Should this behaviour need be different - wseat/bseat and similar controls allow seller to signal that explicitly show who is allowed to bid, albeit on a callout/bid request level, not a specific format

@dmdabbs
Copy link
Contributor

dmdabbs commented Nov 22, 2023

@piwanczak I replied/opened a thread re. biddable on the PR.

@patmmccann
Copy link
Contributor

patmmccann commented Feb 1, 2024

Repeating this comment here:

Object: AuctionGroupIntent
Object: InterestGroupAuctionBuyer

assume that the openrtb request is made by an ssp to a dsp requesting per buyer signals to include on its paapi request; but often an openrtb request is made to a supply platform by a publisher, or between supply platforms. eg (prebid/prebid-server#2411

In this case we'd need a response object with a complete auctionconfig to be submitted or merged with another, not the limited subset of aucionconfig in thie response object here. We're seeing responses vary enormously by prebid implementors

rtbh does this
https://github.com/prebid/Prebid.js/blob/7d3ff582753179b6fb8272122a2d6dea8b0cbab2/modules/rtbhouseBidAdapter.js#L156

magnite puts it here https://github.com/prebid/Prebid.js/blob/7d3ff582753179b6fb8272122a2d6dea8b0cbab2/modules/rubiconBidAdapter.js#L731C32-L731C43

openx puts it here
https://github.com/prebid/Prebid.js/blob/7d3ff582753179b6fb8272122a2d6dea8b0cbab2/modules/openxBidAdapter.js#L105

Openrtb responses need a place for complete configs and we shouldnt publish the per buyer signals response standalone spec until we propose one for the complete object as well

I propose Object: InterestGroupAuctionSeller (PROVISIONAL) with pretty wide open formatting

@patmmccann
Copy link
Contributor

patmmccann commented Feb 5, 2024

I opened #89 to express some feedback we saw Criteo share about ae not matching in place definitions.

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

8 participants