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.

Sign up for GitHub

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

Jump to bottom

[plagyround][server] invocation parameter defaults are missing for most models which leads to inconsistent ux #5390

Open

Tracked by

,

...

Parker-Stafford opened this issue Nov 15, 2024 · 0 comments

Assignees

Labels

c/server

Contributor

Parker-Stafford commented Nov 15, 2024 •

edited

Loading

A couple of things:

invocation parameters are only captured when they are changed on instrumentation - this is fine
invocation parameters don't have defaults which causes numeric ones to be displayed as 0 in the ui. However 0 is not the default from the provider when they are unset, so we need to set defaults (at least for numeric params)

Parker-Stafford mentioned this issue

[playground][dogfooding] initial playground dogfooding check list #5373

Open

38 tasks

github-project-automation bot added this to phoenix

github-project-automation bot moved this to 📘 Todo in phoenix

dosubot bot added the c/ui label

Parker-Stafford changed the title invocation params are only captured if changed, but we don't have defaults plumbed through for most of them for most models and providers leading to potential mismatch between what you see in the ui and what was actually used [plagyround][server] invocation parameter defaults are missing for most models which leads to inconsistent ux

Parker-Stafford assigned anticorrelator

Parker-Stafford added c/server and removed c/ui labels

RogerHYang mentioned this issue

🗺 prompt playground #3435

Open

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment