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

feat: describe graph (PT-188650872) #22

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

Conversation

emcelroy
Copy link
Contributor

@emcelroy emcelroy commented Jan 2, 2025

#188650872

With these changes and related changes to CODAP v3 plus modified AI assistant instructions, the assistant will request an image snapshot of a graph in CODAP when the user asks DAVAI to describe what a graph actually looks like. With the proper CODAP API request, CODAP will send the plugin a base64-encoded string which the plugin can convert to a file and upload to the assistant's file storage for analysis.

To test this out in a browser, you'll need to use a branch CODAP v3 URL, as well as the plugin branch URL with the mode param set to "development", like so:

https://codap3.concord.org/branch/describe-graph-by-text/index.html?sample=mammals&di=https://codap3.concord.org/davai-plugin/branch/describe-graph-by-text/?mode=development

You'll also need to change the assistant to "DAVAI Image Interpretation" in the Developer Options section of the plugin.

@emcelroy emcelroy marked this pull request as ready for review January 6, 2025 01:26
@emcelroy emcelroy marked this pull request as draft January 6, 2025 01:27
@@ -1,38 +1,3 @@
import { AssistantTool } from "openai/resources/beta/assistants";

export const openAiTools: AssistantTool[] = [
Copy link
Contributor Author

Choose a reason for hiding this comment

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

These are no longer needed in the plugin codebase. The tool functions will be created and edited exclusively on the OpenAI platform going forward.

@emcelroy emcelroy requested a review from bgoldowsky January 8, 2025 16:46
@emcelroy emcelroy marked this pull request as ready for review January 8, 2025 16:47
Copy link

@bgoldowsky bgoldowsky left a comment

Choose a reason for hiding this comment

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

Looks good!

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.

3 participants