trjaffe

Results 13 comments of trjaffe

The NAVO Python working group has been looking at PR 470 and have a few questions. (TJ is just speaking for the team.) We hope to see it merged soon,...

``` print(im_discoverer.log_messages) print(im_discoverer.results) ``` Aha, now I see how you're doing it. This is fine, I just didn't realize what you meant. I like things spelled out, as you've done...

The NAVO Python working group has discussed this PR internally and come up with a review. (TJ is just speaking for the team.) Basically, we agree that while we work...

This is fine with me. But it's @bsipocz who has the opinions on how to use astropy's deprecation warning. Brigitta, is there a different type of warning you prefer this...

> I have issues with the alternative as it requires the users to know way more about SIA, SSA, etc. > > So let me phrase it differently: are we...

FWIT, I agree with much of what Brigitta and Adrian said. I don't disagree with Markus' aims, but I don't like the current PR by itself. If we had an...

Oh, I see, this is also how to get nested datalinks, which aren't defined as resources either. I should have realized that. So indeed, all it needs is a small...

(You can assign it to me, since I think it's easy and harmless.)

It should probably just use astropy.io.votable.parse. Sorry, I don't understand what you mean by this > * `examples...execute()` method.

Oh, I see. I was just going to return a list of strings, but it could be a list of TAPQuerys.