-
Notifications
You must be signed in to change notification settings - Fork 13
Update Measure in AI eng #465
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
Conversation
| } | ||
| ); | ||
|
|
||
| Eval('text-match-eval', { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we encourage users to add capability name to metadata?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
IMO capability should be either
- parsed automatically by traversing the trace
- put somewhere more prominent than metadata, which imo is a place for the user to stick whatever they want
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@c-ehrlich we need to write something about cli auth: axiom login, maybe @manototh can help us structure this in a good way. There would be 2 ways to authenticate the SDK:
- Either using CLI, and then you only need to specify a dataset in
axiom.config.ts - Specifying an API token and dataset in axiom.config.ts
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@thesollyz Thanks for this. I would specify API token only because that's what we use in the rest of the AI eng docs anyway.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@manototh not sure what do you mean? CLI auth is a new feature that we want the users to use.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you mean CLI auth is new feature in the context of evals?
If users need to specify the API token anyway in many other places in the setup, I think it would complicate things if we asked them to authenticate the CLI as well. Or am I missing something?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes the CLI auth is a new feature in context of evals. They don't always have to pass a token, its either:
- pass a token
- or login with CLI (then you don't need to pass a token)
Co-authored-by: Islam Shehata <islam@axiom.co>
|
Closing in favor of #473, feel free to re-open if that's wrong |
No description provided.