computed_field properties not showing in tables

[jimkring]

I have a model with some computed_fields and they do not show up automatically when I try to render them in a table, unless I explicitly define the columns of the table (and reference the field by name with a DisplayLookup).

The problem/inconvenience with this, is that now I have to specify every column with a DisplayLookup, just to include the one column that is a computed property. This makes my code less DRY and harder to maintain.

it would be nice if the table logic would find these computed properties and include them when the columns/fields are discovered.

[hasansezertasan]

I believe the main problem is here at the DisplayLookup builder logic:

FastUI/src/python-fastui/fastui/components/tables.py

Lines 46 to 57 in 97c4f07

	if self.columns is None:
	self.columns = [
	display.DisplayLookup(field=name, title=field.title)
	for name, field in data_model_type.model_fields.items()
	]
	else:
	# add pydantic titles to columns that don't have them
	for column in (c for c in self.columns if c.title is None):
	field = data_model_type.model_fields.get(column.field)
	if field and field.title:
	column.title = field.title
	return self

Computed fields are accessible by model.model_computed_fields.items(), check this out:

from pydantic import BaseModel, computed_field


class Rectangle(BaseModel):
    width: int
    length: int

    @computed_field
    @property
    def area(self) -> int:
        return self.width * self.length


data = Rectangle(width=3, length=4)
data.model_fields.items() # Model Fields
data.model_computed_fields.items() # Computed Fields

[hasansezertasan]

I'll work on this tomorrow and try to come up with a proper solution 🤓.

[jimkring]

@hasansezertasan thank you for the good insights and for looking into this enhancement!

[hasansezertasan]

@hasansezertasan thank you for the good insights and for looking into this enhancement!

You're welcome, I'm happy to be able to work on this issue 🤓.

I'd like to have your review if you got time to spare. #321