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

[EuiIcon] Add tokenDimension #7232

Closed
dej611 opened this issue Sep 28, 2023 · 21 comments · Fixed by #7727
Closed

[EuiIcon] Add tokenDimension #7232

dej611 opened this issue Sep 28, 2023 · 21 comments · Fixed by #7727

Comments

@dej611
Copy link
Contributor

dej611 commented Sep 28, 2023

In Kibana we would like to improve support for fields of [time series dimension] type: (https://www.elastic.co/guide/en/elasticsearch/reference/current/tsds.html#time-series-dimension):

elastic/kibana#163323

PR: elastic/kibana#163382

Would be nice to have a new token in EUI for this type of field: https://eui.elastic.co/#/display/icons#tokens,
Visually, I think it should somehow be related to a tag or a label, but this is already taken by the version field type: #5520

@cee-chen
Copy link
Contributor

@MichaelMarcialis or @andreadelrio, do you mind responding to or taking this?

@andreadelrio
Copy link
Contributor

Would be nice to have a new token in EUI for this type of field: https://eui.elastic.co/#/display/icons#tokens, Visually, I think it should somehow be related to a tag or a label, but this is already taken by the version field type: #5520

Given that tokenTag is already taken by the version field type, we could potentially use some of the ideas Elizabeth had for the tokenVersion which we didn't take further because it was decided to use the existing tokenTag.

We could do something like this and instead of the V use a D. @dej611 what do you think?

Frame 3@2x

@dej611
Copy link
Contributor Author

dej611 commented Sep 29, 2023

It would be nice to hint to time series somehow next to the D maybe.

@MichaelMarcialis
Copy link
Contributor

@dej611: A while back, we added TSDB-specific tokens for gauge and counter metric fields. Is this request for a higher-level time series field token? Or would these tokens account for your needs here?

CleanShot 2023-09-29 at 13 21 55

@andreadelrio
Copy link
Contributor

andreadelrio commented Sep 29, 2023

Another option would be using the existing clock icon and passing it to EuiToken.

image

Or alternatively, do a mix of the D with a small clock though at that size it might be hard to tell what each element is (D, clock).

Frame 17

@dej611
Copy link
Contributor Author

dej611 commented Oct 2, 2023

@dej611: A while back, we added TSDB-specific tokens for gauge and counter metric fields. Is this request for a higher-level time series field token? Or would these tokens account for your needs here?

CleanShot 2023-09-29 at 13 21 55

The time series dimension is the complementary part of a time series metric, like a counter or a gauge: i.e. if you have multiple sensors for measuring the temperature somewhere, then temperature would be the gauge metric, while the sensor_id the dimension of the document.

Or alternatively, do a mix of the D with a small clock though at that size it might be hard to tell what each element is (D, clock).

I like this idea, but as you said the clock is big enough to cover the D and make the icon hard to clearly identify.

@JasonStoltz
Copy link
Member

Hey all, any updates on this issue?

@dej611
Copy link
Contributor Author

dej611 commented Oct 17, 2023

I would put this discussion on hold for the moment as there have been few changes in discussion and the outcome of such discussion can have an effect on how and what to define as "time series dimension".
I will keep it posted here and in case will ping to resurrect this issue.

@JasonStoltz
Copy link
Member

Would you mind if we closed this as not planned for now, and we can re-open it later if needed?

@dej611
Copy link
Contributor Author

dej611 commented Oct 17, 2023

sure 👍

@JasonStoltz JasonStoltz closed this as not planned Won't fix, can't repro, duplicate, stale Oct 17, 2023
@dej611
Copy link
Contributor Author

dej611 commented Mar 6, 2024

@JasonStoltz the topic clarified a bit recently, so I think it is time to resurrect this issue. Do you mind to reopen this?

@cee-chen
Copy link
Contributor

cee-chen commented Mar 6, 2024

@dej611 Could we post what the final direction/design for the icon should be before re-opening? Or alternatively, if a designer simply wants to open a PR with tokenDimension, that would be fine as well :)

@dej611
Copy link
Contributor Author

dej611 commented Mar 7, 2024

I think the design from @andreadelrio was on a good track. The clock icon was a bit too big IMHO.

@cee-chen
Copy link
Contributor

cee-chen commented Mar 7, 2024

@andreadelrio @MichaelMarcialis do either of you mind chiming in here before re-opening? Or alternatively @dej611 do you mind writing a bit more about what exactly was clarified on this topic?

@dej611
Copy link
Contributor Author

dej611 commented Mar 11, 2024

The clarification happened more on our side, so no more to add on here.
But I can sum up the whole idea of the semantic for the token: as already stated here a dimension field is the counterpart of a metric field, hence it helps to group by a metric by one or more dimensions.

Let me make an example, say there are log records, metrics are extracted from here, and few dimensions can be defined as host, path, country . Users can then decide to see the metrics overall, broken down by host or by host and country, etc...
This exists only in a time series context.
Only keyword fields can be dimensions for now, but it could be potentially extended the support to other types in the future.

So here's some thinking point:

  • it would be great to have a simple identifiable token shape to quickly spot dimensions vs non-dimensions
  • it would be nice to hint something about time series in the design, but not strongly required
  • if a dimension can be multiple types perhaps it's worth exploring the idea of having some indication of the sub-type (not important now, but perhaps to keep in mind for future iterations)

This is a full brain dump of the problem as best as I can do.

@cee-chen
Copy link
Contributor

Thanks for the context Marco, re-opening. Would appreciate it if a designer could take it from here.

@cee-chen cee-chen reopened this Mar 11, 2024
@cee-chen
Copy link
Contributor

@MichaelMarcialis @andreadelrio Any chance we could get this looked at? Or know a designer we could assign this to?

@MichaelMarcialis
Copy link
Contributor

@andreadelrio, did you wish to continue working with @dej611 on your D + clock token concept? If you've got other higher priorities, let me know and I'd be happy to jump in to help.

@MichaelMarcialis
Copy link
Contributor

Hey, @dej611. I've put together some tokens for this issue. In addition to the requested TSDB dimension field, I've also put together updated concepts for the old TSDB metric counter and gauge fields. The reason for this update was so that we could incorporate some kind of visual connection/relationship between all the TSDB tokens (in this case, via the inclusion of a subscript-styled clock).

Options A and B explore the letter concept you and @andreadelrio proposed earlier. Option C takes a more abstract approach, using a folder icon to convey grouping/categorization, and a bar chart indicating only increasing cumulative values (counter) or values going both up and down (gauge). Let me know if any of these options hit the mark. Thanks!

Actual Size

Option A

Option A

Option B

Option B

Option C

Option C

Zoomed In

Option A

CleanShot 2024-04-12 at 16 17 02

Option B

CleanShot 2024-04-12 at 16 17 22

Option C

CleanShot 2024-04-12 at 16 17 40

@MichaelMarcialis
Copy link
Contributor

@dej611: When you get a moment, would you mind responding to the comment above on which (if any) of the options you'd like to proceed with? Thanks!

@dej611
Copy link
Contributor Author

dej611 commented Apr 29, 2024

Sorry, missed the previous comment.
I think option A makes more sense to me.
I also like option C a lot, but for the I'm a bit afraid the folder icon is not easy to interpret as "dimension": for gauge and counter instead I think the icons work great. Maybe we can mix the dimension icon from A with the other 2 metric icons from C?

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

Successfully merging a pull request may close this issue.

5 participants