Skip to content

Problem with the metadata field (NameError: Field name "metadata" shadows a BaseModel attribute; use a different field name with "alias='metadata'".) #290

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
8 tasks done
MichaelLan opened this issue Mar 29, 2022 · 4 comments
Labels
question Further information is requested

Comments

@MichaelLan
Copy link

First Check

  • I added a very descriptive title to this issue.
  • I used the GitHub search to find a similar issue and didn't find it.
  • I searched the SQLModel documentation, with the integrated search.
  • I already searched in Google "How to X in SQLModel" and didn't find any information.
  • I already read and followed all the tutorial in the docs and didn't find an answer.
  • I already checked if it is not related to SQLModel but to Pydantic.
  • I already checked if it is not related to SQLModel but to SQLAlchemy.

Commit to Help

  • I commit to help with one of those options 👆

Example Code

# database.py

from sqlmodel import Session, create_engine
from core.settings import settings


engine = create_engine(settings.SQLALCHEMY_DATABASE_URL, echo=True)

def get_session():
    with Session(engine) as session:
        yield session


# models.py

from typing import Optional

from sqlmodel import Field, SQLModel


class Companies(SQLModel, table=True):
    avatar: Optional[str] = None
    business_name: Optional[str] = None
    country_id: int = Field(index=True)
    created: str
    crm_id: Optional[str] = None
    id: Optional[str] = Field(default=None, primary_key=True)
    is_active: bool
    is_removed: bool
    metadata_: Optional[str] = Field(default=None, alias="metadata")  # Error: Companies.metadata_ does not exist
    
    # before
    metadata: Optional[str] = Field(default=None)  # Error: NameError

Description

I created a model but this have a field named metadata, is an existing database, so I can't change the fields names, since the SQLModel class also has a field called metadata, when I run the app I get the following error
NameError: Field name "metadata" shadows a BaseModel attribute; use a different field name with "alias='metadata'".

I used the Field class of sqlmodel and assigned the alias but it does not work.
metadata_: Optional[str] = Field(default=None, alias="metadata")
With this the error is that Table.metadata_ field does not exist

Is there any way to create an tabla with the metadata field without giving problems with SQLModel?

Operating System

Linux

Operating System Details

No response

SQLModel Version

0.0.6

Python Version

Python 3.9.5

Additional Context

No response

@MichaelLan MichaelLan added the question Further information is requested label Mar 29, 2022
@MichaelLan
Copy link
Author

I have found the solution using the sa_column argument for the Field class, but the alias argument in the SQLModel class should work.
example code.
metadata_: Optional[str] = Field(sa_column=Column("metadata", String, default=None))

@tcztzy
Copy link

tcztzy commented Aug 1, 2024

Would this problem be fixed? Do we have any walkaround for metadata's validation/serialization currently?

@tcztzy
Copy link

tcztzy commented Aug 1, 2024

Would this problem be fixed? Do we have any walkaround for metadata's validation/serialization currently?

Some research

from pydantic import BaseModel
from pydantic import Field as PydanticField
from sqlmodel import Field, SQLModel


class PydanticModelWithPydanticField(BaseModel):
    metadata_: str = PydanticField(alias="metadata")


class PydanticModelWithSQLModelField(BaseModel):
    metadata_: str = Field(alias="metadata")


class SQLModelWithPydanticField(SQLModel):
    metadata_: str = PydanticField(alias="metadata")


class SQLModelWithSQLModelField(SQLModel):
    metadata_: str = Field(alias="metadata")


if __name__ == "__main__":
    try:
        sql_model_no_suffix = SQLModelWithSQLModelField(metadata="metadata")
        print(f'{sql_model_no_suffix.model_dump(by_alias=False)=}')
        print(f'{sql_model_no_suffix.model_dump(by_alias=True)=}')
    except Exception as e:
        print(f'{e=}')

    sql_model_with_suffix = SQLModelWithSQLModelField(metadata_="metadata")
    print(f'{sql_model_with_suffix.model_dump(by_alias=False)=}')
    print(f'{sql_model_with_suffix.model_dump(by_alias=True)=}')

    sql_model_with_pydantic_field = SQLModelWithPydanticField(metadata="metadata")
    print(f'{sql_model_with_pydantic_field.model_dump(by_alias=False)=}')
    print(f'{sql_model_with_pydantic_field.model_dump(by_alias=True)=}')

    try:
        sql_model_with_pydantic_field_with_suffix = SQLModelWithPydanticField(metadata_="metadata")
        print(f'{sql_model_with_pydantic_field_with_suffix.model_dump(by_alias=False)=}')
        print(f'{sql_model_with_pydantic_field_with_suffix.model_dump(by_alias=True)=}')
    except Exception as e:
        print(f'{e=}')

    pydantic_model_with_pydantic_field = PydanticModelWithPydanticField(metadata="metadata")
    print(f'{pydantic_model_with_pydantic_field.model_dump(by_alias=False)=}')
    print(f'{pydantic_model_with_pydantic_field.model_dump(by_alias=True)=}')

    try:
        pydantic_model_with_pydantic_field_with_suffix = PydanticModelWithPydanticField(metadata_="metadata")
        print(f'{pydantic_model_with_pydantic_field_with_suffix.model_dump(by_alias=False)=}')
        print(f'{pydantic_model_with_pydantic_field_with_suffix.model_dump(by_alias=True)=}')
    except Exception as e:
        print(f'{e=}')

    try:
        pydantic_model_no_suffix = PydanticModelWithSQLModelField(metadata="metadata")
        print(f'{pydantic_model_no_suffix.model_dump(by_alias=False)=}')
        print(f'{pydantic_model_no_suffix.model_dump(by_alias=True)=}')
    except Exception as e:
        print(f'{e=}')

    pydantic_model_with_sql_model_field_with_suffix = PydanticModelWithSQLModelField(metadata_="metadata")
    print(f'{pydantic_model_with_sql_model_field_with_suffix.model_dump(by_alias=False)=}')
    print(f'{pydantic_model_with_sql_model_field_with_suffix.model_dump(by_alias=True)=}')

output of previous code

e=1 validation error for SQLModelWithSQLModelField
metadata_
  Field required [type=missing, input_value={'metadata': 'metadata'}, input_type=dict]
    For further information visit https://errors.pydantic.dev/2.8/v/missing
sql_model_with_suffix.model_dump(by_alias=False)={'metadata_': 'metadata'}
sql_model_with_suffix.model_dump(by_alias=True)={'metadata_': 'metadata'}
sql_model_with_pydantic_field.model_dump(by_alias=False)={'metadata_': 'metadata'}
sql_model_with_pydantic_field.model_dump(by_alias=True)={'metadata': 'metadata'}
e=1 validation error for SQLModelWithPydanticField
metadata
  Field required [type=missing, input_value={'metadata_': 'metadata'}, input_type=dict]
    For further information visit https://errors.pydantic.dev/2.8/v/missing
pydantic_model_with_pydantic_field.model_dump(by_alias=False)={'metadata_': 'metadata'}
pydantic_model_with_pydantic_field.model_dump(by_alias=True)={'metadata': 'metadata'}
e=1 validation error for PydanticModelWithPydanticField
metadata
  Field required [type=missing, input_value={'metadata_': 'metadata'}, input_type=dict]
    For further information visit https://errors.pydantic.dev/2.8/v/missing
e=1 validation error for PydanticModelWithSQLModelField
metadata_
  Field required [type=missing, input_value={'metadata': 'metadata'}, input_type=dict]
    For further information visit https://errors.pydantic.dev/2.8/v/missing
pydantic_model_with_sql_model_field_with_suffix.model_dump(by_alias=False)={'metadata_': 'metadata'}
pydantic_model_with_sql_model_field_with_suffix.model_dump(by_alias=True)={'metadata_': 'metadata'}

Current walk around

Define a new response model inheirt from pydantic.BaseModel instead of SQLModel for fastapi.

conclusion

This problem seems come from SQLModel's Field

@IEH-intekglobal
Copy link

the serialization_alias needs to be set for the FieldInfo. This can be done through the schema_extra argument

metadata_: dict = Field(sa_column=Column("metadata", JSON, nullable=False), default_factory=dict, schema_extra={'serialization_alias': 'metadata'})

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants