<!-- 
RSS generated by JIRA (8.3.4#803005-sha1:1f96e09b3c60279a408a2ae47be3c745f571388b) at Sat Feb 10 16:03:03 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-1122] Investigate poor detectorMap adjustment in r1</title>
                <link>https://pfspipe.ipmu.jp/jira/browse/PIPE2D-1122</link>
                <project id="10002" key="PIPE2D">DRP 2-D Pipeline</project>
                    <description>&lt;p&gt;&lt;a href=&quot;https://pfspipe.ipmu.jp/jira/secure/ViewProfile.jspa?name=shamano&quot; class=&quot;user-hover&quot; rel=&quot;shamano&quot;&gt;Satoshi Hamano&lt;/a&gt; reports that the detectorMap adjustment of 80906r1 has very poor quality.&lt;br/&gt;
Running with &lt;tt&gt;w.2022.46&lt;/tt&gt;, I get the following summary stats:&lt;/p&gt;
&lt;div class=&quot;code panel&quot; style=&quot;border-width: 1px;&quot;&gt;&lt;div class=&quot;codeContent panelContent&quot;&gt;
&lt;pre class=&quot;code-java&quot;&gt;
reduceExposure.adjustDetectorMap INFO: Final result: chi2=107354.909901 dof=44438 xRMS=0.063492 yRMS=0.234515 xSoften=0.040791 ySoften=0.049777 from 34832 lines
reduceExposure.adjustDetectorMap INFO: Stats &lt;span class=&quot;code-keyword&quot;&gt;for&lt;/span&gt; OH: chi2=60810.024394 dof=19260 xRMS=0.081180 yRMS=0.234515 xSoften=0.000000 ySoften=0.345114 from 9630 lines
reduceExposure.adjustDetectorMap INFO: Stats &lt;span class=&quot;code-keyword&quot;&gt;for&lt;/span&gt; OI: chi2=0.000000 dof=0 xRMS=nan yRMS=nan xSoften=0.000000 ySoften=0.000000 from 0 lines
reduceExposure.adjustDetectorMap INFO: Stats &lt;span class=&quot;code-keyword&quot;&gt;for&lt;/span&gt; Trace: chi2=46544.885507 dof=25202 xRMS=0.060551 yRMS=nan xSoften=0.051768 ySoften=0.000000 from 25202 lines
&lt;/pre&gt;
&lt;/div&gt;&lt;/div&gt;
&lt;p&gt;The &quot;Final result&quot; line looks like it might be acceptable at first glance, judging from the &lt;tt&gt;xSoften&lt;/tt&gt; and &lt;tt&gt;ySoften&lt;/tt&gt; values being ~ 0.05; though the yRMS value of 0.23 is concerning, one might think that it&apos;s just reflecting measurement errors. But when looking at the stats for OH lines, we see &lt;tt&gt;ySoften=0.345&lt;/tt&gt; and that&apos;s just unacceptable.&lt;/p&gt;

&lt;p&gt;The expected position of lines using the adjusted detectorMap do not visually coincide with the lines on the image.&lt;/p&gt;</description>
                <environment></environment>
        <key id="23151">PIPE2D-1122</key>
            <summary>Investigate poor detectorMap adjustment in r1</summary>
                <type id="10001" iconUrl="https://pfspipe.ipmu.jp/jira/secure/viewavatar?size=xsmall&amp;avatarId=10515&amp;avatarType=issuetype">Story</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="price">price</reporter>
                        <labels>
                    </labels>
                <created>Thu, 1 Dec 2022 15:49:54 +0000</created>
                <updated>Thu, 2 Feb 2023 19:30:25 +0000</updated>
                            <resolved>Thu, 8 Dec 2022 20:24:37 +0000</resolved>
                                                                        <due></due>
                            <votes>0</votes>
                                    <watches>2</watches>
                                                                <comments>
                            <comment id="31901" author="price" created="Thu, 1 Dec 2022 19:40:25 +0000"  >&lt;p&gt;I&apos;ve become dissatisfied with the current sky linelist, and believe it&apos;s the main reason we&apos;re having trouble adjusting the detectorMap in this case. The main reason is that it merges the Osterbrock and Rousselot lists in an unprincipled manner. The Osterbrock list is linked to observations, while the Rousselot list is a theoretical list without line strengths (the original paper was focussed on NIR observations, and the linelist in the optical appears to have gotten tagged on for use by ESO). Merging the two lists without regard to the line identifications or strengths means that the wavelength of the merged lines can be inaccurate. Furthermore, by merging lines indiscriminately, it becomes difficult to identify lines that are blended.&lt;br/&gt;
I&apos;d like to replace the current sky linelist with a new one that I have constructed solely from the Osterbrock list (at least for the wavelength range it covers). I&apos;ve only merged lines when they are within 0.1 nm and I believe there is reason to expect them to have the same strength (e/f and P1/P2, etc.).&lt;br/&gt;
With the current linelist, running on 80906r1, I get (after a variety of fixes):&lt;/p&gt;
&lt;div class=&quot;code panel&quot; style=&quot;border-width: 1px;&quot;&gt;&lt;div class=&quot;codeContent panelContent&quot;&gt;
&lt;pre class=&quot;code-java&quot;&gt;
reduceExposure.adjustDetectorMap INFO: Final result: chi2=2483923.321630 dof=38826 xRMS=0.055762 yRMS=0.643215 xSoften=0.000000 ySoften=0.510687 from 19425 lines
&lt;/pre&gt;
&lt;/div&gt;&lt;/div&gt;
&lt;p&gt;Notice that the ySoften value of ~0.5 pixels is telling us that there are severe systematic problems with the fit in the spectral dimension (the fit in the spatial dimension is excellent). This is running with an exclusion radius of 3 pixels, so blends should be ignored (but they aren&apos;t). The new linelist gives:&lt;/p&gt;
&lt;div class=&quot;code panel&quot; style=&quot;border-width: 1px;&quot;&gt;&lt;div class=&quot;codeContent panelContent&quot;&gt;
&lt;pre class=&quot;code-java&quot;&gt;
reduceExposure.adjustDetectorMap INFO: Final result: chi2=25275.377851 dof=23700 xRMS=0.057707 yRMS=0.062246 xSoften=0.000000 ySoften=0.000000 from 11862 lines
&lt;/pre&gt;
&lt;/div&gt;&lt;/div&gt;
&lt;p&gt;There are fewer lines (11k vs 19k) used in the fit because we have much more reliable blend identification, and that makes the fit much better: &lt;tt&gt;ySoften=0&lt;/tt&gt;, which means that any errors in the fit are caused by the statistical noise in the data rather than systematics.&lt;/p&gt;</comment>
                            <comment id="31902" author="price" created="Thu, 1 Dec 2022 19:42:35 +0000"  >&lt;p&gt;Attached the notebook that I used to merge the Osterbrock lines and measure the (merged) line intensities. It also contains code to do a cross-correlation of the image and linelist for various detectorMap offsets, which might be helpful in the future.&lt;/p&gt;</comment>
                            <comment id="31906" author="hassan" created="Fri, 2 Dec 2022 20:35:30 +0000"  >&lt;p&gt;I think all the suggested changes and revised linelist look fine to me.&lt;/p&gt;</comment>
                            <comment id="31922" author="price" created="Tue, 6 Dec 2022 15:37:21 +0000"  >&lt;p&gt;Taking this back after review to work on additional detectorMap adjustment updates.&lt;/p&gt;</comment>
                            <comment id="31933" author="price" created="Wed, 7 Dec 2022 21:07:28 +0000"  >&lt;p&gt;Made a few additional changes based on the DRP technical meeting of 2022-12-05 to protect some lines from the exclusion zone.&lt;/p&gt;

&lt;p&gt;Ready for new review.&lt;/p&gt;</comment>
                            <comment id="31941" author="hassan" created="Thu, 8 Dec 2022 19:49:16 +0000"  >&lt;p&gt;Additional changes look fine. Might need to re-phrase one commit comment for drp_stella. See my comments in the pull request.&#160;&lt;/p&gt;</comment>
                            <comment id="31942" author="price" created="Thu, 8 Dec 2022 20:24:29 +0000"  >&lt;p&gt;Reworded that commit, and merged.&lt;/p&gt;</comment>
                            <comment id="32187" author="price" created="Thu, 2 Feb 2023 19:30:25 +0000"  >&lt;p&gt;Origin of the &lt;tt&gt;oh.dat&lt;/tt&gt; file with the Osterbrock lines: &lt;a href=&quot;https://cdsarc.cds.unistra.fr/viz-bin/cat/III/211&quot; class=&quot;external-link&quot; rel=&quot;nofollow&quot;&gt;https://cdsarc.cds.unistra.fr/viz-bin/cat/III/211&lt;/a&gt;&lt;/p&gt;</comment>
                    </comments>
                    <attachments>
                            <attachment id="15449" name="pipe2d-1122.ipynb" size="3864493" author="price" created="Thu, 1 Dec 2022 19:41:10 +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|02qpjd:00r20060i2001</customfieldvalue>

                        </customfieldvalues>
                    </customfield>
                                                                                            <customfield id="customfield_10100" key="com.atlassian.jira.plugin.system.customfieldtypes:userpicker">
                        <customfieldname>Reviewers</customfieldname>
                        <customfieldvalues>
                            <customfieldvalue>hassan</customfieldvalue>

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