You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This is an example I went through today noting locations where the current API is inadequate (it almost certainly parallels @low-sky 's signal-id package). Take note of the TODO items:
@akleroy@low-sky If you're going to be putting effort into signal-id this round, could you prep up an example like this one? I think this is the exact target case for signal-id; I'm selecting pixels based on signal-to-noise in one or more cubes with neighbor significance checking etc.
Sure thing a rich example is high on our list - I've been testing on Tw
Hydra SV data; I'll see if I can work a wget in there that snags the
relevant file (it's not sooooo big, but still seems like poor form to check
in in). Erik has added some basic tests for the noise methodology in his
last check in.
The biggest snafu I've seen so far was horrible crashing in the old way to
calculate the plate scale. I didn't figure out what caused it but it was
actually segfaulting ipython. I put something simpler in its place and have
been okay so far.
@akleroyhttps://github.com/akleroy @low-skyhttps://github.com/low-skyIf you're going to be putting effort into signal-id this round, could you
prep up an example like this one? I think this is the exact target case for
signal-id; I'm selecting pixels based on signal-to-noise in one or more
cubes with neighbor significance checking etc.
—
Reply to this email directly or view it on GitHubhttps://github.com//issues/104#issuecomment-44407339
.
There's a chance that old segfault issue may have been trouble with the compiled code in WCS; there was a bug we found a few weeks back in which the WCSLIB C code tried to access a memory location that wasn't allocated. Anyway, sounds like you solved it.
This is an example I went through today noting locations where the current API is inadequate (it almost certainly parallels @low-sky 's
signal-id
package). Take note of theTODO
items:The text was updated successfully, but these errors were encountered: