-
Notifications
You must be signed in to change notification settings - Fork 50
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
2.6-202501 Public Comment Feedback #121
Comments
If you open the PR, main...develop, it is a lot easier to comment on the pr in the right place instead of the comments here |
This continues one of the largest problems in openrtb, that cat, pagecat, bcats etc can only have one taxonomy. There is no upgrade path when a new taxonomy is released and you will be stuck on whatever the initial taxonomy is forever. Supply will never be able to stop sending the first taxonomy for fear of being misunderstood by non-adopters of the new taxonomy. Demand will never release support for a new taxonomy for lack of supply adoption. This was solved in site.content.data and user.data like this: https://github.com/InteractiveAdvertisingBureau/openrtb/blob/main/extensions/community_extensions/segtax.md#example and i think we should repeat that pattern for genres.
where segtax 8 is Content Taxonomy 3.1 and I am proposing the taxtype field if someone wants to use that. Or we could make a new segtax enum 9 that is the genre subtax of content tax 3.1 we also need a pr on https://github.com/InteractiveAdvertisingBureau/openrtb/blob/main/extensions/community_extensions/segtax.md or adcom define segtax 8 If we're absolutely stuck on not using the site.content.data pattern, i would propose we have an array of genre objects, not gtax and genre as their own field with only one entry allowed |
There is a PR against AdCom to enumerate Content 3.1 as 9 (Ad Product 2.0 is 8) |
Fair enough, but I don't love using site.content.data. Would you be open to making gtax and genre elements of an object to pass multiple taxos as needed? We could potentially tackle doing the same for site/app.cattax for bcat and cat next year if it works. |
I 100% agree with @patmmccann that we should not repeat the mistakes of the past and create a spec that only allows one taxonomy to be sent in a request. As an industry, we are stuck using IAB 1.0 taxonomy with cat, pagecat, bcats because there's no feasible migration path when it's not possible to send fields using both the old and new versions in a bid request. |
Is there any proposed language to deal with |
Please comment on this issue with feedback on updates to the OpenRTB updates in Public Comment
OpenRTB Updates:
gtax
andgenres
attributes in Object: ContentTaxonomy Updates:
genres
attributeThe text was updated successfully, but these errors were encountered: