BUG: Default Values in Model Are Incorrectly Set as 'ExpressionProxy' Objects

[shreypandey]

When defining default values in a JsonModel, these values are unexpectedly being set as ExpressionProxy objects instead of the intended default values.

Expected Behavior:

Default values defined in the model should be directly set as the specified values.

Actual Behavior:

Default values are replaced with ExpressionProxy objects.

Steps to reproduce:

When running the following code:

import datetime
from typing import Optional

from redis_om import JsonModel, Field


class Customer(JsonModel):
    first_name: str
    last_name: str
    email: str
    join_date: datetime.date
    age: int = Field(default=18)
    bio: Optional[str] = "Super dope"


class Child(Customer):
    is_new: bool = Field(default=True)


andrew = Child(
    first_name="Andrew",
    last_name="Brookins",
    email="andrew.brookins@example.com",
    join_date=datetime.date.today(),
)

print("Age Field: ", andrew.age)
print("Bio Field: ", andrew.bio)
print("Model Field:", andrew.model_dump())

produces the following output:

Age Field:  <redis_om.model.model.ExpressionProxy object at 0x7fb70dcdf8e0>
Bio Field:  <redis_om.model.model.ExpressionProxy object at 0x7fb70db8c160>
Model Field: {'pk': '01JCV9FESAVVNAVBG2XJSTQGNN', 'first_name': 'Andrew', 'last_name': 'Brookins', 'email': 'andrew.brookins@example.com', 'join_date': datetime.date(2024, 11, 17), 'age': <redis_om.model.model.ExpressionProxy object at 0x7fb70dcdf8e0>, 'bio': <redis_om.model.model.ExpressionProxy object at 0x7fb70db8c160>, 'is_new': True}

Additional Notes:

However, replacing JsonModel by pydantic BaseModel works fine

import datetime
from typing import Optional

from pydantic import BaseModel
from redis_om import Field


class Customer(BaseModel):
    first_name: str
    last_name: str
    email: str
    join_date: datetime.date
    age: int = Field(default=18)
    bio: Optional[str] = "Super dope"


class Child(Customer):
    is_new: bool = Field(default=True)


andrew = Child(
    first_name="Andrew",
    last_name="Brookins",
    email="andrew.brookins@example.com",
    join_date=datetime.date.today(),
)

print("Age Field: ", andrew.age)
print("Bio Field: ", andrew.bio)
print("Model Field:", andrew.model_dump())

This code produces the following output:

Age Field:  18
Bio Field:  Super dope
Model Field: {'first_name': 'Andrew', 'last_name': 'Brookins', 'email': 'andrew.brookins@example.com', 'join_date': datetime.date(2024, 11, 17), 'age': 18, 'bio': 'Super dope', 'is_new': True}

Environment

• Python: 3.10.15
• Implementation: CPython
• Platform: linux
• redis-om version: 0.3.3
• pydantic version: 2.9.2

[slorello89]

interesting, it appears to have something to do with the inheretence of the model, will have to look into it.

[sethbuff]

We are having this issue as well. It's going to prevent us from upgrading to pydantic 2.0

[slorello89]

For some odd reason in child classes it looks like some context WRT the defaults is being lost, I opened a PR to fix it (it just applies the defaults).

[shreypandey]

Hi @slorello89, can you please tell us when the PR will be merged? It's preventing us to integrate redis in our project.

[XChikuX]

Seems like #612 is related to this. Looking for a fix soon.

[sethbuff]

@slorello89 We are coming back to this but it appears the release hasn't been cut that includes this. Is there any reason we haven't released this yet?