Skip to content

feat: link data to graph image for description (DAVAI-38) #46

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

emcelroy
Copy link
Contributor

@emcelroy emcelroy commented May 23, 2025

DAVAI-38

When an graph image snapshot is sent to the LLM, we now include data about the attributes on the graph for additional context. This helps the LLM describe the graph image better, especially for large datasets where points on a graph may overlap.

@emcelroy emcelroy force-pushed the DAVAI-38-link-data-to-graph-image-for-description branch from ad7453c to 80160ee Compare May 29, 2025 15:04
@emcelroy emcelroy force-pushed the DAVAI-38-link-data-to-graph-image-for-description branch from 80160ee to 5feb83c Compare May 30, 2025 21:15
@emcelroy emcelroy marked this pull request as ready for review May 30, 2025 21:20
@emcelroy emcelroy requested a review from Copilot May 30, 2025 21:21
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds functionality to link attribute data for graphs to the image snapshot description, enhancing the context provided to the LLM.

  • Added an interface for graph attribute data.
  • Introduced new flow functions (getAttributeData and getGraphAttrData) to fetch and combine graph attribute information.
  • Modified the message sending logic to include the graph attribute context.

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

Successfully merging this pull request may close these issues.

1 participant