-
Notifications
You must be signed in to change notification settings - Fork 151
docs: Document embedded chart visualizations in AI chat (for PR #8122) #8203
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
base: main
Are you sure you want to change the base?
Conversation
- Update AI chat documentation to reflect new chart visualization capabilities - Document chatCharts feature flag in project configuration - Add details about automatic chart type selection and when charts are generated - Include examples of enabling chat with visualizations - List all available chart types (line, area, bar, stacked bar, donut, combo, heatmap) Related to PR #8122 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
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.
@doclaude make changes, since a PR owner cant request changes, see if comments work.
EDIT: triggered, yay but skipped aww
|
Trying regular comment |
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.
@doclaude do your thing.
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.
@doclaude do your thing.
|
Branch: Check the workflow logs for details. |
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.
@doclaude do you thing.
|
✅ DoClaude processed all inline review comments 📝 Changes have been applied to the branch: 🔄 Review comments have been marked as addressed 📚 The agent analyzed the review comments and may have updated Review the commit and push additional changes if needed. |
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.
@doclaude
|
✅ DoClaude processed all inline review comments 📝 Changes have been applied to the branch: 🔄 Review comments have been marked as addressed 📚 The agent analyzed the review comments and may have updated Review the commit and push additional changes if needed. |
|
hm as you can see its not doing the best in small changes hmmm |
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.
@doclaude
|
✅ DoClaude processed all inline review comments 📝 Changes have been applied to the branch: 🔄 Review comments have been marked as addressed 📚 The agent analyzed the review comments and may have updated Review the commit and push additional changes if needed. |
This reverts commit 8152c2d.
|
yeah its being weird for small changes. But i guess thats expected with AI. I guess will require a human to verify still. worked better in the GCS one |
|
Adding @djbarnwal as he's the PR owner of #8122 |
|
@royendo, where in docs can i find this? |
Summary
This PR documents the new embedded chart visualization feature in AI chat introduced in PR #8122.
Changes
Updated AI chat documentation (
docs/docs/explore/ai-chat.md):Updated project configuration documentation (
docs/docs/build/project-configuration.md):chatChartsfeature flag to the list of available flagsKey Features Documented
chatChartsflag controls this functionality (default: false)Related
🤖 Generated with Claude Code