get_step_handler() needs 3 parameters

llm-edge / hal-9100

Edge full-stack LLM platform. Written in Rust

MIT License

371 stars 30 forks source link

Open ClancyDennis opened 8 months ago

ClancyDennis commented 8 months ago

In my testing---

line 16 api-communication/run_steps pub async fn get_step_handler( Path((run_id, step_id)): Path<(String, String)>,

goes to

pub async fn get_step_handler( Path((thread_id, run_id, step_id)): Path<(String, String, String)>,

fixes empty row returned by sqlx

n.b. still unable to run chainlit-cookbook/openai-assistants example (unwrapping None value)

louis030195 commented 8 months ago

Thx for testing this @ClancyDennis ! Steps was implemented recently, still experimental. Any contribution highly appreciated!