Skip to content

fix(cache): clean cache on object deletion#114

Merged
jourdain merged 1 commit into
masterfrom
clean-cache
May 21, 2026
Merged

fix(cache): clean cache on object deletion#114
jourdain merged 1 commit into
masterfrom
clean-cache

Conversation

@Jo-Byr
Copy link
Copy Markdown
Contributor

@Jo-Byr Jo-Byr commented May 21, 2026

Remove cached properties of deleted objects to reduce cache size and prevent cache corruption

Fixes lookup table errors referenced in this issue

Prevent a cache registered for a previous object being used for another object with a same key.

@Jo-Byr Jo-Byr requested a review from jourdain May 21, 2026 06:49
@Jo-Byr Jo-Byr self-assigned this May 21, 2026
Copy link
Copy Markdown
Member

@finetjul finetjul left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you explain how to reproduce cache corruption (a unit test if systematically reproducible would be even better)

@Jo-Byr
Copy link
Copy Markdown
Contributor Author

Jo-Byr commented May 21, 2026

Could you explain how to reproduce cache corruption (a unit test if systematically reproducible would be even better)

The only reproducible test I have is in trame-pyvista, should I still document it here ?

@finetjul
Copy link
Copy Markdown
Member

The only reproducible test I have is in trame-pyvista, should I still document it here ?

Referencing the PyVista issue is a must.
Explaining what happens (in commit message or PR) would be great.

Remove cached properties of deleted objects to reduce cache size
and prevent cache corruption
@Jo-Byr
Copy link
Copy Markdown
Contributor Author

Jo-Byr commented May 21, 2026

@jourdain the pipeline fails partly due to an mismatch error between images, but I think the test function is incorrect, I opened a PR to fix it

@jourdain jourdain merged commit d9de578 into master May 21, 2026
2 of 3 checks passed
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.

3 participants