astropy / astropy/astroquery

ENH: add `cache` kwarg to irsa.Most() or turn off caching altogether

Open
#3,476 1 comment 0 reactions 0 assignees View on GitHub
bug enhancement ipac.irsa
Dominant language
Python
Stars
791
Forks
451
Avg merge
1d 3h
Merged PRs (30d)
4

Description

Currently the Most module relies on some behaviour inherited from the baseclass to use caching which gives no options to the end user to turn on or off caching (it's on by global config default via the `cache_conf.cache_active` default).

In practice this causes troubles as the D9 region files are deleted after a short time thus causing troubles with the cached queries.

It could also be an acceptable alternative to turn off caching for this module.

cc @jonesmg for additional details as he was the one noticing this issue while developing IRSA tutorial notebooks.

Contributor guide

Open the contributing guide

Research direction

Start by reading the irsa.Most implementation and the inherited baseclass caching behavior, including the cache_conf.cache_active setting. Reproduce the D9-region cache failure if possible, then determine whether the module needs a cache kwarg or caching disabled; done means the selected behavior works for IRSA tutorial notebook use without stale deleted-region queries.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
api, performance
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.