expectedparrot / edsl

Design, conduct and analyze results of AI-powered surveys and experiments. Simulate social science and market research with large numbers of AI agents and LLMs.
https://docs.expectedparrot.com
MIT License
99 stars 16 forks source link

Consider deprecating Cache.remote #686

Closed onmyraedar closed 1 month ago

onmyraedar commented 1 month ago

We added remote caching logic in Jobs.run(), but the Cache object itself has an existing remote attribute that I believe was used to handle the earlier iteration of remote caching.

What should we do about Cache.remote? Should we delete that attribute and its logic, or keep it and explicitly override the value of Cache.remote with the value from the Coop settings?

Ex.: if the user passes q.run(cache=Cache(remote=False)), but remote caching is set to true on Coop, we would need to override.

johnjosephhorton commented 1 month ago

We should - I'll re-factor it out.