Unverified Commit d573aead authored by Michael Goin's avatar Michael Goin Committed by GitHub
Browse files

[Bugfix] Don't log OpenAI field aliases as ignored (#11378)


Signed-off-by: default avatarmgoin <michael@neuralmagic.com>
parent 995f5623
...@@ -46,7 +46,15 @@ class OpenAIBaseModel(BaseModel): ...@@ -46,7 +46,15 @@ class OpenAIBaseModel(BaseModel):
@classmethod @classmethod
def __log_extra_fields__(cls, data): def __log_extra_fields__(cls, data):
if isinstance(data, dict): if isinstance(data, dict):
extra_fields = data.keys() - cls.model_fields.keys() # Get all class field names and their potential aliases
field_names = set()
for field_name, field in cls.model_fields.items():
field_names.add(field_name)
if hasattr(field, 'alias') and field.alias:
field_names.add(field.alias)
# Compare against both field names and aliases
extra_fields = data.keys() - field_names
if extra_fields: if extra_fields:
logger.warning( logger.warning(
"The following fields were present in the request " "The following fields were present in the request "
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment