C. E. Brasseur

Results 70 comments of C. E. Brasseur
trafficstars

@orionlee No it doesn't make that PR obsolete. Because the MAST DBs update so regularly as new data comes in all the time, I had wanted this underlying work done...

@bsipocz I'm marking this ready for review. I haven't written the documentation yet, but the tests are in place, and I can do the docs concurrent with working on the...

@bsipocz I'm working on implementing @eerovaher's suggested changes. I'm almost there, and when I've got them in place I will integrate https://github.com/astropy/astroquery/pull/1578.

@barentsen Oh weird, I will take a look. Thanks for noticing. 😝

I'm swamped through Monday, but should have time to test this later next week.

@christinahedges I will try to make time next week to look at this PR. I am also in general concerned with the lack of an expiration on the cache, which...

@christinahedges Yeah, I think ideally the cache will have a default timeout on the short scale, the user will have the ability to set it to a user specified value,...

@bsipocz I will try to look at the caching situation overall next week then, and gather what needs to be done and make a plan. 😃

@bsipocz @christinahedges Opened WIP PR https://github.com/astropy/astroquery/pull/1634 to address the general caching situation. Will look at this PR next.

@bsipocz Excellent Idea, I'll work on this next.