-
Notifications
You must be signed in to change notification settings - Fork 2
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
Explicitly vs. automatically selected dimensions #51
Comments
Hey, thanks!
I'll try something and we'll then discuss the details in a PR. |
Note that querying for labels on all dimensions makes everything much slower, so I wonder if there would be a better way to do this. E.g. by only querying for labels in |
Could you please tell us more about this? Would running |
I meant that currently, This is probably related to #47 … adding labels to the query unfortunately makes it much slower. BTW, we're currently also always setting all potential languages on the entrypoint, e.g. |
Yeah adding labels definitely makes things slower, and yes adding more languages makes it even slower. I think not fetching labels for automatically selected dimensions and using dimensions IRIs as keys would solve most of the issue. Users could fetch dimensions and their labels independently and possibly cache them.
@ktk what do you think about this, is it possible to declare the languages somewhere? |
Hi!
I just noticed a discrepancy between how explicitly and automatically selected dimensions are handled, and another aspect which makes the automatic selects less-than-useful.
Point 2 could actually neatly be solved by not generating keys from the label but by using the dimension IRI. If behavior in point 1 would be consistent (i.e. labels present for auto-selects), this would actually remove the need to explicitly select dimensions at all.
For example:
If IRIs are used as keys, the argument to
.select()
could be simply an array of components or just their IRIs instead of having to specify binding names myself (which is also dangerous since these are not slugified!).The text was updated successfully, but these errors were encountered: