openllmetry icon indicating copy to clipboard operation
openllmetry copied to clipboard

πŸš€ Feature: Add `gen_ai.request.model` to data points attributes of meter

Open minimAluminiumalism opened this issue 5 months ago β€’ 4 comments

Which component is this feature for?

OpenAI Instrumentation

πŸ”– Feature description

Now the data points attributes of meter only has the attribute gen_ai.response.model but doesn't have gen_ai.request.model. Taking gen_ai.server.time_to_first_token as an example:

Data point attributes:
     -> gen_ai.system: Str(openai)
     -> gen_ai.response.model: Str(moonshot-v1-128k)
     -> gen_ai.operation.name: Str(chat)
     -> server.address: Str(https://api.moonshot.cn/v1/)
     -> stream: Bool(true)

Maybe we should also add it?

🎀 Why is this feature needed ?

Enrich meter attributes

✌️ How do you aim to achieve this?

Similar to the existing attr gen_ai.reponse.model

πŸ”„οΈ Additional Information

No response

πŸ‘€ Have you spent some time to check if this feature request has been raised before?

  • [x] I checked and didn't find similar issue

Are you willing to submit PR?

Yes I am willing to submit a PR!

minimAluminiumalism avatar Jul 16 '25 10:07 minimAluminiumalism

Requesting for permission to work on this issue. Should take at most 2 weeks, no more than that.

jyotirmayc avatar Sep 07 '25 17:09 jyotirmayc

Requesting for permission to work on this issue. Should take at most 2 weeks, no more than that.

@nirga Please take a look since I don’t have the permission to assign :)

minimAluminiumalism avatar Sep 07 '25 17:09 minimAluminiumalism

@minimAluminiumalism do you want to join as a collaborator?

nirga avatar Sep 07 '25 21:09 nirga

@minimAluminiumalism do you want to join as a collaborator?

Sure :)

minimAluminiumalism avatar Sep 08 '25 02:09 minimAluminiumalism