<!-- 
RSS generated by JIRA (8.3.4#803005-sha1:1f96e09b3c60279a408a2ae47be3c745f571388b) at Sat Feb 10 16:03:32 JST 2024

It is possible to restrict the fields that are returned in this document by specifying the 'field' parameter in your request.
For example, to request only the issue key and summary append 'field=key&field=summary' to the URL of your request.
-->
<rss version="0.92" >
<channel>
    <title>PFS-JIRA</title>
    <link>https://pfspipe.ipmu.jp/jira</link>
    <description>This file is an XML representation of an issue</description>
    <language>en-us</language>    <build-info>
        <version>8.3.4</version>
        <build-number>803005</build-number>
        <build-date>13-09-2019</build-date>
    </build-info>


<item>
            <title>[PIPE2D-1156] fluxTable masks too severe</title>
                <link>https://pfspipe.ipmu.jp/jira/browse/PIPE2D-1156</link>
                <project id="10002" key="PIPE2D">DRP 2-D Pipeline</project>
                    <description>&lt;p&gt;By looking at the original data I noticed that in noisy parts of the spectra some isolated pixels remain unflagged, and with values that do not seem to be coherent with the (current) noise estimate of the spectrum. These pixels are used by PIPE1D just as others and can induce wrong measurement if their value is not correct. This problem might be solved by the correction of the noise estimate though. &lt;span class=&quot;image-wrap&quot; style=&quot;&quot;&gt;&lt;img src=&quot;https://pfspipe.ipmu.jp/jira/secure/attachment/15514/15514_NovRun_mask_621_635nm.png&quot; style=&quot;border: 0px solid black&quot; /&gt;&lt;/span&gt;&lt;/p&gt;</description>
                <environment></environment>
        <key id="23228">PIPE2D-1156</key>
            <summary>fluxTable masks too severe</summary>
                <type id="1" iconUrl="https://pfspipe.ipmu.jp/jira/secure/viewavatar?size=xsmall&amp;avatarId=10503&amp;avatarType=issuetype">Bug</type>
                                            <priority id="10000" iconUrl="https://pfspipe.ipmu.jp/jira/images/icons/priorities/medium.svg">Normal</priority>
                        <status id="10002" iconUrl="https://pfspipe.ipmu.jp/jira/images/icons/statuses/generic.png" description="The issue is resolved, reviewed, and merged">Done</status>
                    <statusCategory id="3" key="done" colorName="green"/>
                                    <resolution id="10000">Done</resolution>
                                        <assignee username="price">price</assignee>
                                    <reporter username="vlebrun">vlebrun</reporter>
                        <labels>
                    </labels>
                <created>Wed, 1 Feb 2023 12:17:47 +0000</created>
                <updated>Wed, 1 Mar 2023 18:36:10 +0000</updated>
                            <resolved>Wed, 1 Mar 2023 18:36:10 +0000</resolved>
                                                                        <due></due>
                            <votes>0</votes>
                                    <watches>7</watches>
                                                                <comments>
                            <comment id="32163" author="price" created="Wed, 1 Feb 2023 15:13:09 +0000"  >&lt;p&gt;Could you please provide instructions to reproduce this?&lt;/p&gt;</comment>
                            <comment id="32164" author="vlebrun" created="Wed, 1 Feb 2023 15:17:19 +0000"  >&lt;p&gt;I used Topcat on the FluxTable extension, plotted all the pixels (in red), then selected all pixels with mask=0 and overplotteed them (in blue)&lt;/p&gt;

&lt;p&gt;the file is&#160;pfsObject-00010-00001-1,1-009954820000a57e-006-0x6e85dbfdd0e3cf63.fits&lt;/p&gt;

&lt;p&gt;from the GE_COS1 dataset (pfsDesignId=0x608f5cd5e916bf12: visit=83244..83249 )&lt;/p&gt;

&lt;p&gt;Actually I had no clear expectation on such a bad part of the spectrum, an I&apos;d wait for the issues on the noise evaluation before focusing on this one, it might just be that it will be solved then&lt;/p&gt;</comment>
                            <comment id="32165" author="price" created="Wed, 1 Feb 2023 15:20:26 +0000"  >&lt;p&gt;On what file? How was the data processed? What version was used? What do you expect the result to be, and how does this differ from that?&lt;/p&gt;

&lt;p&gt;&lt;a href=&quot;https://github.com/Subaru-PFS/drp_doc/blob/master/pipelineUser/help.rst&quot; class=&quot;external-link&quot; rel=&quot;nofollow&quot;&gt;https://github.com/Subaru-PFS/drp_doc/blob/master/pipelineUser/help.rst&lt;/a&gt;&lt;/p&gt;</comment>
                            <comment id="32315" author="price" created="Wed, 1 Mar 2023 03:21:34 +0000"  >&lt;p&gt;At least part of the problem is that &lt;tt&gt;mask=0&lt;/tt&gt; excludes good pixels as well as bad. On this coadd, we&apos;ve &lt;tt&gt;OR&lt;/tt&gt;-ed the input masks while excluding bad pixels, so &lt;tt&gt;CR&lt;/tt&gt; means that there was a cosmic-ray at that pixel in one of the inputs, but we&apos;ve excluded it in the process of coadding (but maybe we didn&apos;t do a great job with neighbouring pixels). We&apos;ve also added a mask plane (which, due to a bug, wasn&apos;t showing up; and wasn&apos;t being calculated very well) called &lt;tt&gt;OVERLAP&lt;/tt&gt; which indicates pixels from one arm that have better coverage by another arm. So you should ignore pixels with &lt;tt&gt;(mask &amp;amp; NO_DATA) != 0&lt;/tt&gt;. &lt;/p&gt;

&lt;p&gt;I&apos;ve fixed a couple of things in the fluxTable, and made the plot with the correct masking, and I think it looks much better.&lt;/p&gt;

&lt;p&gt;&lt;span class=&quot;image-wrap&quot; style=&quot;&quot;&gt;&lt;img src=&quot;https://pfspipe.ipmu.jp/jira/secure/attachment/15543/15543_pip2d-1156.png&quot; width=&quot;1000&quot; style=&quot;border: 0px solid black&quot; /&gt;&lt;/span&gt;&lt;/p&gt;</comment>
                            <comment id="32316" author="vlebrun" created="Wed, 1 Mar 2023 08:17:20 +0000"  >&lt;p&gt;looks definitely better indeed. Is this new masking ( ignore pixels with &lt;tt&gt;(mask &amp;amp; NO_DATA) != 0 ) going to be definite, or just to wait for your fix ?&lt;/tt&gt;&lt;/p&gt;</comment>
                            <comment id="32323" author="price" created="Wed, 1 Mar 2023 15:21:02 +0000"  >&lt;p&gt;In general, you should never expect that all mask planes indicate something bad, so each time you use a mask you need to ask what mask planes you care about.&lt;/p&gt;</comment>
                            <comment id="32324" author="price" created="Wed, 1 Mar 2023 18:36:10 +0000"  >&lt;p&gt;Merged.&lt;/p&gt;</comment>
                    </comments>
                    <attachments>
                            <attachment id="15514" name="NovRun_mask_621_635nm.png" size="34817" author="vlebrun" created="Wed, 1 Feb 2023 12:13:55 +0000"/>
                            <attachment id="15543" name="pip2d-1156.png" size="569073" author="price" created="Wed, 1 Mar 2023 03:09:14 +0000"/>
                    </attachments>
                <subtasks>
                    </subtasks>
                <customfields>
                                                <customfield id="customfield_10500" key="com.atlassian.jira.plugins.jira-development-integration-plugin:devsummary">
                        <customfieldname>Development</customfieldname>
                        <customfieldvalues>
                            
                        </customfieldvalues>
                    </customfield>
                                                                                                                                                                                                            <customfield id="customfield_10010" key="com.pyxis.greenhopper.jira:gh-lexo-rank">
                        <customfieldname>Rank</customfieldname>
                        <customfieldvalues>
                            <customfieldvalue>0|02qph7:07f</customfieldvalue>

                        </customfieldvalues>
                    </customfield>
                                                                                                                        <customfield id="customfield_10005" key="com.pyxis.greenhopper.jira:gh-sprint">
                        <customfieldname>Sprint</customfieldname>
                        <customfieldvalues>
                                <customfieldvalue id="162">2DDRP-2023 A</customfieldvalue>

                        </customfieldvalues>
                    </customfield>
                                                                                                                                                    </customfields>
    </item>
</channel>
</rss>