-
Notifications
You must be signed in to change notification settings - Fork 396
Add Patronus OTEL Exporter #341
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
|
@hersheybar Can you fix the DCO check so we can get this merged? |
Allow AIQToolkit agentic traces to export logs to the Patronus AI backend. Update the 'simple_calculator' example with a config file which talks to the Patronus endpoint. Signed-off-by: Hersh Mehta <hersh@patronus.ai>
7e014df to
5a93aa6
Compare
|
Hello @mdemoret-nv : |
|
@hersheybar Thanks for the updates. DCO is now passing. Would you mind enabling contributions to your PR from maintainers so I can update the docs to match in a single PR? Just need to select this option on the PR: |
|
@mdemoret-nv : I do not see that checkbox anywhere (at the bottom, or on the right hand side tab). Note that the string: "Allow edits and acess..." also doesn't appear on the page source. Am I missing something? |
|
Im not sure what is going on. I'll just make the docs updates in a second PR and merge this one. |
|
Awesome, thanks @mdemoret-nv for the quick responses! |
|
Yes. One second. Need to rerun CI after each commit. |
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.
Will update docs in a second PR
|
/ok to test 283e454 |
|
Looks like we've been stuck on the |
|
/merge |
Allow AIQToolkit agentic traces to export logs to the Patronus AI backend. Update the 'simple_calculator' example with a config file which talks to the Patronus endpoint. Closes NVIDIA#340 ## By Submitting this PR I confirm: - I am familiar with the [Contributing Guidelines](https://github.com/NVIDIA/AIQToolkit/blob/develop/docs/source/resources/contributing.md). - We require that all contributors "sign-off" on their commits. This certifies that the contribution is your original work, or you have rights to submit it under the same license, or a compatible license. - Any contribution which contains commits that are not Signed-Off will not be accepted. - When the PR is ready for review, new or existing tests cover these changes. - When the PR is ready for review, the documentation is up to date with these changes. Authors: - Hersh Mehta (https://github.com/hersheybar) Approvers: - Michael Demoret (https://github.com/mdemoret-nv) URL: NVIDIA#341
Allow AIQToolkit agentic traces to export logs to the Patronus AI backend. Update the 'simple_calculator' example with a config file which talks to the Patronus endpoint. Closes NVIDIA#340 ## By Submitting this PR I confirm: - I am familiar with the [Contributing Guidelines](https://github.com/NVIDIA/AIQToolkit/blob/develop/docs/source/resources/contributing.md). - We require that all contributors "sign-off" on their commits. This certifies that the contribution is your original work, or you have rights to submit it under the same license, or a compatible license. - Any contribution which contains commits that are not Signed-Off will not be accepted. - When the PR is ready for review, new or existing tests cover these changes. - When the PR is ready for review, the documentation is up to date with these changes. Authors: - Hersh Mehta (https://github.com/hersheybar) Approvers: - Michael Demoret (https://github.com/mdemoret-nv) URL: NVIDIA#341


Description
Allow AIQToolkit agentic traces to export logs to the Patronus AI backend. Update the 'simple_calculator' example with a config file which talks to the Patronus endpoint.
Closes #340
By Submitting this PR I confirm: