Opened 12 years ago
Closed 12 years ago
#142 closed defect (fixed)
Inconsistencies in searches with MW vs subimages
Reported by: | MatthewWhiting | Owned by: | MatthewWhiting |
---|---|---|---|
Priority: | normal | Milestone: | Release-1.2 |
Component: | Searching | Version: | 1.1.13 |
Severity: | normal | Keywords: | |
Cc: |
Description
A series of emails with BiQing?, initiated by the following:
Hi, Matthew, I wrote a paper (and about to submit) that is based on Duchamp so the following issue is really worried me. I used Duchamp to search on a cube by specifying the MW channels. I got 838 sources. Then, I decided to extract a subcube (using miriad imsub) that excludes those MW channels so the cube size is reduced and flags can be given from Duchamp if there is any source has velocity width cut off near the edge. Basically, both ran on the exact same channels and same parameters. I got 598 sources on the subcube. This shouldn't happen. Note: segmentation fault didn't occur on these cubes since they are smaller. The cube is in K unit rather than Jy/beam. The cubes were run with fix threshold. I am sending you the output and input from both runs. Perhaps you can help looking into this? Thanks, BiQing
Investigate and see if it is still a problem with 1.1.14
Change History (3)
comment:1 Changed 12 years ago by
Status: | new → assigned |
---|
comment:2 Changed 12 years ago by
Here is my email to Bi Qing from last week, detailing the results of my investigation on gismo:
Hi BiQing, Sorry for not getting back to you sooner - I was away on holidays last week, and have had a somewhat interrupted week with sick kids and other things. Here's a summary of what I understand is going on: * The reconstruction needs a bit of careful understanding. The reason you get different results between the full cube and the full cube with flagSubsection=true is due to the way the reconstruction behaves. One key feature is that boundaries are dealt with by assuming reflection. For a given point, calculating the large-scale wavelet coefficients involves multiplying the filter coefficients by points a long way away. Since we reflect at the boundary, if that boundary is in a different place, the filter coefficient will be multiplied by a different pixel value, giving a different result - *even if the point in question is well within your desired subsection*. The differences will in general be small, but they are there, and are probably more obvious for your data where there is large- scale structure. However, this is at odds with your comment in your last email on the weekend: > to save you some time. I ran the original cube and the subcube. The output of both wavelet reconstruction cubes are > identical. Thus, the problem is something else as we suspected earlier. I'm not sure I understand that though, as that is not what I'm finding (I trim the full reconstructed cube to the same dimensions as the subcube, and I get differences, explained by the above reasoning). * The reason you get a difference between the flagSubsection=true case and the IMSUB case appears to be that IMSUB has written a BLANK=-1 keyword to the FITS header. This is then affecting the reconstruction (pixels that are deemed to be BLANK are left alone in the reconstruction). I've checked this in two ways: 1) I made a similar subcube using casapy, and got the same results (subject to the offset in the z-direction) as for the flagSubsection=true case - the reconstructed cubes here are identical. 2) I copied the imsub.fits file and changed the BLANK keyword so that it wouldn't get recognised. I also found this gave the same results as flagSubsection=true (save for the z-offset). * The MW flagging does not seem to be important - this is done at the searching stage, and is just a way of not looking at the relevant channels. They still get reconstructed though. Does all this make sense? We can discuss further next week if you like - the monday meeting won't happen due to the public holiday, but we can talk another time if need be. Cheers, Matt.
Anyway, the upshot is that I think I understand what is going on now. I'm currently running a test with the latest code to check that there aren't any further issues cropping up, and that we get the same results as described above.
comment:3 Changed 12 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Happy enough to close this, after the large number of tests run on giant (see evernote notes on this) and on the solution of #153.
Note: See
TracTickets for help on using
tickets.
Further emails:
Me:
BiQing?:
Me again: