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

job chat: claude issues #97

Open
josephjclark opened this issue Sep 13, 2024 · 11 comments · Fixed by #118 · May be fixed by #130
Open

job chat: claude issues #97

josephjclark opened this issue Sep 13, 2024 · 11 comments · Fixed by #118 · May be fixed by #130
Assignees

Comments

@josephjclark
Copy link
Collaborator

An issue to share issues found in Claude's responses in job chat

@josephjclark
Copy link
Collaborator Author

josephjclark commented Sep 13, 2024

#141 - Bad use of fn()(state) in the each each callback

image

@josephjclark josephjclark self-assigned this Sep 13, 2024
@josephjclark
Copy link
Collaborator Author

Claude doesn't seem to know what adaptor is being used

image

@josephjclark

This comment has been minimized.

@josephjclark
Copy link
Collaborator Author

josephjclark commented Sep 18, 2024

From Aisha: don't always assume the answer is job code - could just be regular js

This was my question to the AI: Given a JSON data, and given a list of columns in an sql table, give me an insert query. 

The AI assistant immediately assumed i wanted an OpenFn code for the request i made. I just wanted it to structure a json object into an sql insert query format so i can try it on the db. I'm not sure if the AI assuming I wanted an OpenFn code by default was an intended feature but wanted to point that out

#160

@josephjclark
Copy link
Collaborator Author

We've had two reports that the AI doesn't know what the user's job code looks like.

This could be a lightning issue but more likely it's the model being wild

Sorry I don't have any evidence to hand here

@josephjclark
Copy link
Collaborator Author

josephjclark commented Jan 16, 2025

Collections! I'll bet there's nothing in the prompt to explain the collections API. This is a great case where a good RAG would really help

See #143

@hunterachieng hunterachieng added this to v2 Jan 17, 2025
@github-project-automation github-project-automation bot moved this to New Issues in v2 Jan 17, 2025
@josephjclark
Copy link
Collaborator Author

josephjclark commented Jan 22, 2025

From Jack: My experience with the AI Assistant so far is that it has been pretty verbose/excessive. ,Any time I run through what I've made with Taylor he usually re-writes it with a much simpler expression which achieves what I actually want to do, without the bells and whistles (which I guess Claude wants to give us out of the box)?

#157

@josephjclark
Copy link
Collaborator Author

josephjclark commented Jan 23, 2025

Question: Please open a previous session

Answer: I apologize, but I don't have access to any previous sessions or conversations. I'm an AI language model designed to provide information and assistance based on the current conversation only. Each interaction starts fresh.

Is there something specific from a previous conversation that you'd like to discuss or ask about? I'd be happy to help you with any questions or topics related to OpenFn job writing or DHIS2 integration to the best of my current knowledge and capabilities.


The assistant is a bit out of line here. It doesn't understand its context properly - not every interaction starts fresh. Actually, we use sessions with memory, so Claude does have some recall and context.

Also, while it's true that Claude can't open a session itself, it could (in a perfect world) tell users how to do it.

Related: what if users ask "what can you see?". #156

We should improve Claude's context about how it is being used in our app.

@josephjclark
Copy link
Collaborator Author

josephjclark commented Jan 23, 2025

Terrible handling of DHIS2. Here's some sample code it gave:

// Configuration for source DHIS2 instance
alterState(state => ({
  ...state,
  configuration: {
    username: 'source_username',
    password: 'source_password',
    baseUrl: 'https://source.dhis2.org/api'
  }
}));

// Fetch data from source DHIS2
get('resourceType', {
  // Add query parameters here
});

// Transform data if needed
fn(state => {
  const sourceData = state.data;
  // Add your data transformation logic here
  return { ...state, transformedData: sourceData };
});

// Configuration for destination DHIS2 instance
alterState(state => ({
  ...state,
  configuration: {
    username: 'destination_username',
    password: 'destination_password',
    baseUrl: 'https://destination.dhis2.org/api'
  }
}));

// Send data to destination DHIS2
create('resourceType', state => state.transformedData);

The big issues here are:

@aleksa-krolls
Copy link
Member

hey @hunterachieng I'm not clear on what the status is here/ next steps. Is there any AI work you need to wrap up in this coming sprint?

@hunterachieng
Copy link
Contributor

@aleksa-krolls I have worked on an AI issue that is currently under review. I was to tackle this after that was reviewed but we can slot it for another day this week.

@theroinaochieng theroinaochieng moved this from New Issues to DevX Backlog in v2 Feb 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: DevX Backlog
4 participants