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

Legacy native implementation: why is aspect_ratios set to an object in fromOrtbNativeRequest? #9922

Closed
msmeza opened this issue May 9, 2023 · 2 comments · Fixed by #9923
Closed
Assignees
Labels

Comments

@msmeza
Copy link
Contributor

msmeza commented May 9, 2023

Type of issue

Question

Description

In the function fromOrtbNativeRequest in native.js, img.aspect_ratios is set to an object, but a global search for aspect_ratios in this repository shows that most occurrences are expected to be of type array. Why is aspect_ratios set to an object and not an array in fromOrtbNativeRequest?

@dgirardi
Copy link
Collaborator

dgirardi commented May 9, 2023

I believe this is simply a bug (paging @musikele just in case you disagree)

@musikele
Copy link
Contributor

This is what can be found in previous documentation:

image

Then, in the example, it is an array:
image

Honestly i don't remember what pushed me to treat as an object. Probably unit tests, or probably the fact that in ORTB you cannot specify multiple aspect ratios...

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

Successfully merging a pull request may close this issue.

3 participants