docs: make should_cache docstring implementation-agnostic

Remove prescriptive filtering suggestions - let implementations
decide their own caching logic based on their use case.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
This commit is contained in:
Deep Mehta 2026-01-29 19:52:06 +05:30
parent d755f7ca19
commit 4afa80dc07

View File

@ -121,9 +121,8 @@ class CacheProvider(ABC):
Called before on_lookup (value=None) and on_store (value provided).
Return False to skip external caching for this node.
Common filters:
- By class_type: Only cache expensive nodes (KSampler, VAEDecode)
- By cost/benefit: Skip if download time > compute time
Implementations can filter based on context.class_type, value size,
or any custom logic. Use estimate_value_size() to get value size.
Default: Returns True (cache everything).
"""