-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[Obs ai assistant][ESQL] Visualizes a query #173076
Conversation
@@ -32,7 +32,6 @@ const containerClassName = css` | |||
`; | |||
|
|||
const conversationListContainerName = css` | |||
min-width: 250px; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Trying to make it more responsive, otherwise the poor push flyout looks awful
export const visualizeESQLFunction = { | ||
name: 'visualize_query', | ||
description: | ||
'Use this function to visualize charts for ES|QL queries. DO NOT run the lens function after.', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ℹ️ I had to instruct it to not run the lens function after, not sure if this is the best way to do it but it worked
keptLayerIds: string[]; | ||
} | ||
|
||
export const getLensAttributes = ({ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@stratoula reuse this is Lens
Half of this code has been merged (at the Visualizations side) and I decided to take a slightly different route so I am closing it in favor of this which is cleaner #174677 and in sync with the latest main. |
Summary
This is a PoC, do not expect to be awesome :)
This PR
With clicking the apply button, the new chart configuration is saved to the conversation
User can save the ES|QL chart on a dashboard. From there they can continue editing the chart (we also display the same inline editing flyout giving a seamless experience)
Discussion points: