Skip to content

cache mesh cell_sizes - #5317

Open
leo-collins wants to merge 2 commits into
mainfrom
leo/cell-sizes-cache
Open

cache mesh cell_sizes#5317
leo-collins wants to merge 2 commits into
mainfrom
leo/cell-sizes-cache

Conversation

@leo-collins

Copy link
Copy Markdown
Contributor

Decorate cell_sizes with a cached_property_until the coordinates change, and deprecate clear_cell_sizes

@leo-collins

Copy link
Copy Markdown
Contributor Author

@connorjward Interestingly, the clear_cell_sizes function deleted the cell_size attribute, which doesn't exist, and not the cell_sizes attribute. So it appears this function wasn't doing anything. It has been like this since 2018.

@connorjward

Copy link
Copy Markdown
Contributor

That's fun. We should probably still deprecate it properly in case it was getting called (pointlessly) by users.

@leo-collins
leo-collins force-pushed the leo/cell-sizes-cache branch from 3d033cd to 4642e22 Compare August 3, 2026 20:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants