Skip to content

Adsk contrib - Add method to clear processor cache#1791

Merged
doug-walker merged 9 commits into
AcademySoftwareFoundation:mainfrom
autodesk-forks:adsk_contrib/add-method-to-clear-processor-cache
May 4, 2023
Merged

Adsk contrib - Add method to clear processor cache#1791
doug-walker merged 9 commits into
AcademySoftwareFoundation:mainfrom
autodesk-forks:adsk_contrib/add-method-to-clear-processor-cache

Conversation

@cedrik-fuoco-adsk
Copy link
Copy Markdown
Contributor

@cedrik-fuoco-adsk cedrik-fuoco-adsk commented Apr 21, 2023

This PR adds to the public API a method to clear the Processor cache of a Config instance. This is useful in situations where LUT files on disk have changed after a Processor was created.

Signed-off-by: Cédrik Fuoco <cedrik.fuoco@autodesk.com>
Adding test for the cache enable/disable cycle
Adding test to test clearProcessorCache

Signed-off-by: Cédrik Fuoco <cedrik.fuoco@autodesk.com>
…nimize PRs)

Signed-off-by: Cédrik Fuoco <cedrik.fuoco@autodesk.com>
Tweak unit tests
FIx implementation for clearProcessorCache

Signed-off-by: Cédrik Fuoco <cedrik.fuoco@autodesk.com>
reverting zlib version and will do it in another PR

Signed-off-by: Cédrik Fuoco <cedrik.fuoco@autodesk.com>
Signed-off-by: Cédrik Fuoco <cedrik.fuoco@autodesk.com>
Signed-off-by: Cédrik Fuoco <cedrik.fuoco@autodesk.com>
Signed-off-by: Cédrik Fuoco <cedrik.fuoco@autodesk.com>
Copy link
Copy Markdown
Collaborator

@remia remia left a comment

Choose a reason for hiding this comment

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

Thanks for the fix @cedrik-fuoco-adsk, I also encountered this few months back but had forgotten about it, my temporary workaround was to clone the config using createEditableCopy() which also clears the processor cache.

@doug-walker doug-walker merged commit f0a181a into AcademySoftwareFoundation:main May 4, 2023
@doug-walker doug-walker deleted the adsk_contrib/add-method-to-clear-processor-cache branch May 4, 2023 01:19
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