Skip to content

Conversation

@liustve
Copy link
Contributor

@liustve liustve commented Nov 7, 2024

Description of changes:

contract tests for new gen_ai inference parameter added in

e8c96ae#diff-20c2ca1cb28cda6e03ec0cb986933b2abd103bee39995ad232cc2e8c2d23e4aaR368

image

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

feat: set up gen ai inference attributes for foundational text models…
- added contract tests for amazon titan and anthropic claude models
- added support to compare float attributes
@liustve liustve requested a review from a team as a code owner November 7, 2024 23:06
@yiyuan-he
Copy link
Contributor

Contract test case failing in workflow. Could you help take a look?

https://github.com/aws-observability/aws-otel-js-instrumentation/actions/runs/11806377445/job/32890847301?pr=119

@liustve
Copy link
Contributor Author

liustve commented Nov 14, 2024

Contract test case failing in workflow. Could you help take a look?

https://github.com/aws-observability/aws-otel-js-instrumentation/actions/runs/11806377445/job/32890847301?pr=119

oh the only contract test failing is the cohere_command_r model which should work once your PR goes through @yiyuan-he

@yiyuan-he
Copy link
Contributor

Contract test case failing in workflow. Could you help take a look?
https://github.com/aws-observability/aws-otel-js-instrumentation/actions/runs/11806377445/job/32890847301?pr=119

oh the only contract test failing is the cohere_command_r model which should work once your PR goes through @yiyuan-he

Sounds good, thanks for confirming!

@yiyuan-he yiyuan-he self-requested a review November 21, 2024 23:00
@mxiamxia mxiamxia merged commit 27c0f80 into aws-observability:main Nov 22, 2024
9 checks passed
@liustve liustve deleted the dev branch December 23, 2024 18:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants