Skip to content

add minor fixes to JS/TS Exercise#6

Open
davbizer93 wants to merge 1 commit intomainfrom
david-testing
Open

add minor fixes to JS/TS Exercise#6
davbizer93 wants to merge 1 commit intomainfrom
david-testing

Conversation

@davbizer93
Copy link
Copy Markdown
Collaborator

Went through the JS/TS Exercise E2E and have 1-2 suggestions. Below are my notes

--> no deployment found when passing resource group
--> Should be like in solution with model name :

constructor(modelName: 'sap-rpt-1-small' | 'sap-rpt-1-large' = 'sap-rpt-1-small') {

// Initialize the SDK client

// By default, it uses 'sap-rpt-1-small'

this.client = new RptClient(modelName)

}

--> Update Step 2 (currently not working)
--> Update Step 3 to include payload in buildGraph():

const workflow = new StateGraph<AgentState>({

channels: {

payload: null,

suspect_names: null,

appraisal_result: null,

messages: null,

},

});

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