-
Notifications
You must be signed in to change notification settings - Fork 809
Open
Description
Probably this is a simple change, I think just 3 sem convs were added:
Lines 170 to 188 in bd3c1f2
| if use_latest_semconvs: | |
| if "seed" in generation_config: | |
| attributes[GenAIAttributes.GEN_AI_REQUEST_SEED] = ( | |
| generation_config.seed | |
| ) | |
| if "candidate_count" in generation_config: | |
| attributes[GenAIAttributes.GEN_AI_REQUEST_CHOICE_COUNT] = ( | |
| generation_config.candidate_count | |
| ) | |
| if "response_mime_type" in generation_config: | |
| if generation_config.response_mime_type == "text/plain": | |
| attributes[GenAIAttributes.GEN_AI_OUTPUT_TYPE] = "text" | |
| elif generation_config.response_mime_type == "application/json": | |
| attributes[GenAIAttributes.GEN_AI_OUTPUT_TYPE] = "json" | |
| else: | |
| attributes[GenAIAttributes.GEN_AI_OUTPUT_TYPE] = ( | |
| generation_config.response_mime_type | |
| ) | |
But need to double check what AI sem convs were added that are missing from the old "non-experimental" versions of these instrumentations..
Metadata
Metadata
Assignees
Labels
No labels