pyvo icon indicating copy to clipboard operation
pyvo copied to clipboard

cachedataset is a misleading name for the function

Open RayPlante opened this issue 11 years ago • 3 comments

Many users have construed that the cachedataset() function (of the SIARecord and SSARecord classes) involves the use of a special storage area on disk that can avoid having to re-download a file multiple times in a session. Instead, this function simply saves the dataset to a specified file. We should consider changing the name to something like save_dataset(). This will become more important as we integrate more with astropy and take advantage of its caching capabilities.

RayPlante avatar Oct 11 '13 18:10 RayPlante

Have you thought about adding a verbose or download status flag that would print the size (in kB / MB / GB) of the file being downloaded and some sort of periodic update? The download status input could be used to set how often updates occur, something like:

Downloading http://irsa.ipac.caltech.edu/ibe/data/wise/allsky/4band_p3am_cdd/20/2029/2029p469_ab41/2029p469_ab41-w1-int-3.fits
64.0 MB ......

If there are no such plans, I'll have to come up with something to output the file size before I download something unwieldy large.

jimaples avatar Nov 12 '13 13:11 jimaples

If there are no such plans, I'll have to come up with something to output the file size before I download something unwieldy large.

I would like to see that. thanks!

RayPlante avatar Jan 17 '14 16:01 RayPlante

Oh, I misread the last line of your comment--I thought you had something in mind. This should be very easy to do.

RayPlante avatar Jan 17 '14 16:01 RayPlante