Xee
Xee copied to clipboard
Implement lazy loading to defer metadata RPCs until data access time …
Hi @alxmrs,
Added lazy loading (issue #44):
lazy_load=True option
Defers metadata RPCs → faster large dataset opens
Backward compatible + tested
can you review my PR?
Hi @alxmrs I've addressed all the feedback from the review:
- Pass
lazy_loadas an argument toget_info()instead of usinggetattr - Replaced the magic number with proper key sets to check cache status
- Extracted property fetching into a separate helper method
_fetch_collection_properties() - Used
time.perf_counter()instead oftime.time()in tests for more accurate timing - Refactored the caching system to use a more consistent approach
Let me know if there's anything else I should adjust!
Thanks @pvrraju and @alxmrs! Let's revisit this after #275 is in a pre or stable release.