C. E. Brasseur

Results 70 comments of C. E. Brasseur
trafficstars

@bsipocz @keflavich Could I get your opinions on the big picture questions I mentioned above?

@D-arioSpace Looking into the `neocc.query_object(name='433', tab='observations')` error, and I can't reproduce it. I get a table that looks superficially fine to me as output, can you describe what you're seeing?

Thanks @keflavich! To your first point, I can't tell which paradigm you are advocating. Always returning a list?

@D-arioSpace At this point, everything substantive is finished, it's just a matter of getting all the tests passing and then the final sign off from @bsipocz/@keflavich. The main test issue...

@D-arioSpace I've got all the tests passing and addressed the ones of @bsipocz's comments that I can. All that needs to happen now is that you address the outstanding comments,...

@barentsen This is interesting. You are right, tesscut does not have a mechanism for specifying camera. I did not anticipate that there would ever be overlap between cameras (most of...

@keflavich @bsipocz You want me to change the caching mechanism from pickle to something safer yes?

@keflavich My experience (primarily with MAST, but I've been thinking about the construction while working on the new NEOCC service too) matches with yours in that the async versions have...

I also prefer option number 2 for the reasons @bsipocz outlines. Additionally I think that option 1 will lead to cases where there isn't one clear home for certain classes....

I'm getting two failures when I run the remote tests, both the same: `AttributeError: 'Table' object has no attribute 'pformat_all'`