Opened 11 years ago
Closed 11 years ago
#198 closed defect (fixed)
Velocity width measurement failing
| Reported by: | MatthewWhiting | Owned by: | MatthewWhiting |
|---|---|---|---|
| Priority: | normal | Milestone: | Release-1.5 |
| Component: | Code base | Version: | 1.4 |
| Severity: | normal | Keywords: | |
| Cc: |
Description
In my selavy testing with the M31 cube, one of the workers was failing with the mask optimisation. It turns out that the calculation of the velocity width is failing, with inf values being returned.
See if this can be replicated with Duchamp and fixed.
Attachments (1)
Change History (3)
Changed 11 years ago by
| Attachment: | duchamp.in added |
|---|
comment:1 Changed 11 years ago by
| Milestone: | → Release-1.5 |
|---|---|
| Status: | new → assigned |
The problem lay in the velocity width calculation. There are a lot of masked pixels in this cube, and although the calculation of the integrated flux has a mask included, this is set to allow all pixels.
I have patched the code so that this is now fixed - the param is passed to the integrated flux function, and from there to the velocity widths, where it is used to generate the mask.
Seems to work - need to test with Selavy.
Have attached the parameter file used for testing.
comment:2 Changed 11 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
Works fine. Closing.
Parameter set for testing with the M31 cube