Helicone Community Page

Updated 3 months ago

Parameter Mismatch and Undefined References in Request View

Hello,

I've found an issue within the Request View. Here's a detailed breakdown:

  1. In the Request View ( https://www.helicone.ai/requests ), there seems to be an inconsistency with parameter handling that results in referencing an undefined parameter. Specifically:
  1. Given that the Model is "undefined", the following code will always return "GPT3Builder": https://github.com/Helicone/helicone/blob/b6a0b8355abf8d499ea197a4ab6f149de5b56d54/web/components/templates/requestsV2/builder/requestBuilder.tsx#L18
  1. When response_body.choices exists, the code tries to reference this.response.response_body?.choices[0].text. However, the actual response of azure openai models is this.response.response_body?.choices[0].message: https://github.com/Helicone/helicone/blob/b6a0b8355abf8d499ea197a4ab6f149de5b56d54/web/components/templates/requestsV2/builder/GPT3Builder.tsx#L44
I hope this helps in rectifying the issues. Please let me know if you need further information.

Br,
Harry
Attachments
02-not-working-with-prompts-history-02.png
02-not-working-with-prompts-history-01.png
01-working-without-prompts-history.png
h
a
11 comments
I havent looked into diffs in gpt35 between openai and azure. They might have different response format
Hey ! Will get to this as soon as I can. A bit busy this morning with some personal stuff. Will update here when a fix is in!
can you check now? Can you also add me into your org if you don't mind me doing some preview deployment testing for your account in the future?
It's not fixed yet but I added you as member to our organization πŸ™‚
Thanks! Taking a quick look in a sec
Hi ! This bug is still not fixed yet πŸ™‚ Do you need more help?
hi ! sorry for the delay! looking into it right now
merging in a PR now
definitely need to rewrite our "default" case here. I patched in a temporary fix and the team will follow up with a more robust fix for the future.
Thank you for the response! I've checked it on my end and it's fixed πŸ™‚ Sounds great that you improve code architecture as well πŸ˜‰
Add a reply
Sign up and join the conversation on Discord