search

Deci-AI / data-gradients

Computer Vision dataset analysis
Apache License 2.0
293 stars 33 forks source link

Feature/sg 945 move summary before adding adapter #167

Closed Louis-Dupont closed 1 year ago

Louis-Dupont commented 1 year ago

What was changed

The core here is to replace in the DataConfig use_cache with

    cache_filename: Optional[str] = None
    cache_dir: Optional[str] = None

AND to load the cache on instantiation. This requires some minor logic changes but None of them should affect users in anyway.

Motivation

Currently, the cache from DataConfig is loaded by the AnalysisManager, i.e. after instantiation. And it is the AnalysisManager that provides the path to the cache. We want to be able to load it easily on instantiation, in order to be able to load and use it directly anywhere, and not just from within the AnalysisManager