<!-- 
RSS generated by JIRA (8.3.4#803005-sha1:1f96e09b3c60279a408a2ae47be3c745f571388b) at Sat Feb 10 15:55:23 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-611] reduceArc should also fix the x-direction of the DetectorMap</title>
                <link>https://pfspipe.ipmu.jp/jira/browse/PIPE2D-611</link>
                <project id="10002" key="PIPE2D">DRP 2-D Pipeline</project>
                    <description>&lt;p&gt;As discussed on telecons June 19 and June 22, `reduceArc` function that modifies DetectorMap at the moment only `fixes` the&#160;DetectorMap in the wavelength direction (y-direction), but it does not modify the&#160;DetectorMap in the x-direction (perpendicular to the wavelength direction). This causes 2D PSF subtractions to be very poor and it is not possible to analyze the quality of the subtraction.&#160;&lt;/p&gt;

&lt;p&gt;(I have left story points empty as I have no good idea of the scope of this work)&lt;/p&gt;</description>
                <environment></environment>
        <key id="14530">PIPE2D-611</key>
            <summary>reduceArc should also fix the x-direction of the DetectorMap</summary>
                <type id="3" iconUrl="https://pfspipe.ipmu.jp/jira/secure/viewavatar?size=xsmall&amp;avatarId=10518&amp;avatarType=issuetype">Task</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="ncaplar">ncaplar</reporter>
                        <labels>
                    </labels>
                <created>Mon, 22 Jun 2020 23:57:22 +0000</created>
                <updated>Mon, 4 Jan 2021 20:22:56 +0000</updated>
                            <resolved>Sun, 28 Jun 2020 16:04:01 +0000</resolved>
                                                                        <due></due>
                            <votes>0</votes>
                                    <watches>2</watches>
                                                                <comments>
                            <comment id="17371" author="price" created="Wed, 24 Jun 2020 20:44:38 +0000"  >&lt;p&gt;&lt;tt&gt;reduceArc.py&lt;/tt&gt; already has a &lt;tt&gt;doUpdateCenters&lt;/tt&gt; config parameter, but it uses the fiberTrace! I&apos;ll add something to tweak it using the arc lines also.&lt;/p&gt;

&lt;p&gt;Choosing which feature to use should be left to the user, as the software can&apos;t know whether the user will prefer the centers from the arc or the quartz.&lt;/p&gt;</comment>
                            <comment id="17386" author="price" created="Fri, 26 Jun 2020 22:02:20 +0000"  >&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;
(lsst-scipipe) pprice@tiger2-sumire:/tigress/pprice/pipe2d-611 $ generateCommands.py /projects/HSC/PFS/LAM/ --calib /projects/HSC/PFS/LAM/CALIB-price --rerun price/pipe2d-611 --blocks=pipe2d_611 -j 10 pipe2d-611.yaml calibs.sh --devel --force

(lsst-scipipe) pprice@tiger2-sumire:/tigress/pprice/pipe2d-611 $ detrend.py /projects/HSC/PFS/LAM --calib /projects/HSC/PFS/LAM/CALIB-price --rerun price/pipe2d-611 --id visit=21515 arm=r --no-versions

(lsst-scipipe) pprice@tiger2-sumire:/tigress/pprice/pipe2d-611 $ bootstrapDetectorMap.py /projects/HSC/PFS/LAM --calib /projects/HSC/PFS/LAM/CALIB-price --rerun price/pipe2d-611 --flatId visit=21122 arm=r --arcId visit=21515 arm=r

bootstrap INFO: Matched 370 lines
bootstrap INFO: Median difference from detectorMap: -9.974619,-5.693824 pixels
bootstrap INFO: Fit 193/228 points, rms: x=0.043026 y=0.119466 total=0.088148 pixels
bootstrap INFO: Updating detectorMap...
bootstrap INFO: Median difference from detectorMap: -10.189670,-5.453311 pixels
bootstrap INFO: Fit 120/142 points, rms: x=0.101351 y=0.097857 total=0.085120 pixels
bootstrap INFO: Updating detectorMap...

(lsst-scipipe) pprice@tiger2-sumire:/tigress/pprice/pipe2d-611 $ cp /projects/HSC/PFS/LAM/rerun/price/pipe2d-611/DETECTORMAP/pfsDetectorMap-021515-r1.fits /projects/HSC/PFS/LAM/CALIB-price/DETECTORMAP/pfsDetectorMap-000000-r1.fits

Looks pretty good! We&apos;re going to have to introduce an artificial shift, to demonstrate we can undo it...

&amp;gt;&amp;gt;&amp;gt; &lt;span class=&quot;code-keyword&quot;&gt;import&lt;/span&gt; astropy.io.fits
&amp;gt;&amp;gt;&amp;gt; ff = astropy.io.fits.open(&lt;span class=&quot;code-quote&quot;&gt;&quot;/projects/HSC/PFS/LAM/CALIB-price/DETECTORMAP/pfsDetectorMap-000000-r1.fits&quot;&lt;/span&gt;)
&amp;gt;&amp;gt;&amp;gt; ff[3].header[&lt;span class=&quot;code-quote&quot;&gt;&quot;EXTNAME&quot;&lt;/span&gt;]
&lt;span class=&quot;code-quote&quot;&gt;&apos;CENTER&apos;&lt;/span&gt;
&amp;gt;&amp;gt;&amp;gt; ff[3].data.value += 1.0
&amp;gt;&amp;gt;&amp;gt; ff.writeto(&lt;span class=&quot;code-quote&quot;&gt;&quot;/projects/HSC/PFS/LAM/CALIB-price/DETECTORMAP/pfsDetectorMap-000000-r1.fits&quot;&lt;/span&gt;, clobber=True)

(lsst-scipipe) pprice@tiger2-sumire:/tigress/pprice/pipe2d-611 $ displayDetectorMap.py /projects/HSC/PFS/LAM --calib /projects/HSC/PFS/LAM/CALIB-price --rerun price/pipe2d-611 --id visit=21515 arm=r --no-versions

(lsst-scipipe) pprice@tiger2-sumire:/tigress/pprice/pipe2d-611 $ reduceArc.py /projects/HSC/PFS/LAM --calib /projects/HSC/PFS/LAM/CALIB-price --rerun price/pipe2d-611 --id visit=21515 arm=r --no-versions

reduceArc INFO: Tweaking fiber 650 xCenter by -1.232491 (+/- 0.049559)
reduceArc INFO: Tweaking fiber 587 xCenter by -1.166069 (+/- 0.029186)
reduceArc INFO: Tweaking fiber 525 xCenter by -1.219642 (+/- 0.038179)
reduceArc INFO: Tweaking fiber 464 xCenter by -1.191371 (+/- 0.045484)
reduceArc INFO: Tweaking fiber 401 xCenter by -1.206986 (+/- 0.057679)
reduceArc INFO: Tweaking fiber 339 xCenter by -1.186496 (+/- 0.110483)
reduceArc INFO: Tweaking fiber 255 xCenter by -1.190592 (+/- 0.118263)
reduceArc INFO: Tweaking fiber 192 xCenter by -1.098192 (+/- 0.114480)
reduceArc INFO: Tweaking fiber 63 xCenter by -1.195772 (+/- 0.094612)
reduceArc INFO: Tweaking fiber 2 xCenter by -1.110896 (+/- 0.140032)

(lsst-scipipe) pprice@tiger2-sumire:/tigress/pprice/pipe2d-611 $ displayDetectorMap.py /projects/HSC/PFS/LAM --calib /projects/HSC/PFS/LAM/CALIB-price --rerun price/pipe2d-611 --id visit=21515 arm=r --no-versions -c frame=2
&lt;/pre&gt;
&lt;/div&gt;&lt;/div&gt;</comment>
                            <comment id="17387" author="ncaplar" created="Fri, 26 Jun 2020 22:03:22 +0000"  >&lt;p&gt;Looks good!&lt;/p&gt;</comment>
                            <comment id="17388" author="price" created="Fri, 26 Jun 2020 22:03:54 +0000"  >&lt;p&gt;The arc-based tweak works, as demonstrated in the attached figures.&lt;/p&gt;

&lt;p&gt;Before tweak (with an artificial 1 pixel offset):&lt;br/&gt;
&lt;span class=&quot;image-wrap&quot; style=&quot;&quot;&gt;&lt;img src=&quot;https://pfspipe.ipmu.jp/jira/secure/attachment/12710/12710_before.png&quot; style=&quot;border: 0px solid black&quot; /&gt;&lt;/span&gt;&lt;/p&gt;

&lt;p&gt;After tweak:&lt;br/&gt;
&lt;span class=&quot;image-wrap&quot; style=&quot;&quot;&gt;&lt;img src=&quot;https://pfspipe.ipmu.jp/jira/secure/attachment/12709/12709_after.png&quot; style=&quot;border: 0px solid black&quot; /&gt;&lt;/span&gt;&lt;/p&gt;</comment>
                            <comment id="17389" author="price" created="Fri, 26 Jun 2020 22:15:20 +0000"  >&lt;p&gt;&lt;a href=&quot;https://pfspipe.ipmu.jp/jira/secure/ViewProfile.jspa?name=ncaplar&quot; class=&quot;user-hover&quot; rel=&quot;ncaplar&quot;&gt;ncaplar&lt;/a&gt;: note the addition of a new script, &lt;tt&gt;displayDetectorMap.py&lt;/tt&gt;, which I hope will make it easier to see the quality of the detectorMaps relative to the arcs. See the above log for details on how to use it.&lt;/p&gt;</comment>
                            <comment id="17390" author="ncaplar" created="Fri, 26 Jun 2020 22:18:19 +0000"  >&lt;p&gt;Ok. Looking forward to it. This is tested on simulation and not on the real data, right?&lt;/p&gt;

&lt;p&gt;&#160;&lt;/p&gt;</comment>
                            <comment id="17391" author="price" created="Fri, 26 Jun 2020 23:58:55 +0000"  >&lt;p&gt;The above is LAM data.&lt;/p&gt;</comment>
                            <comment id="17401" author="price" created="Sun, 28 Jun 2020 16:04:01 +0000"  >&lt;p&gt;Merged to master yesterday, while Jira was down.&lt;/p&gt;</comment>
                    </comments>
                <issuelinks>
                            <issuelinktype id="10000">
                    <name>Blocks</name>
                                            <outwardlinks description="blocks">
                                        <issuelink>
            <issuekey id="14134">PIPE2D-490</issuekey>
        </issuelink>
                            </outwardlinks>
                                                        </issuelinktype>
                    </issuelinks>
                <attachments>
                            <attachment id="12709" name="after.png" size="162395" author="price" created="Fri, 26 Jun 2020 22:01:28 +0000"/>
                            <attachment id="12710" name="before.png" size="162516" author="price" created="Fri, 26 Jun 2020 22:01:29 +0000"/>
                            <attachment id="12711" name="pipe2d-611.yaml" size="671" author="price" created="Fri, 26 Jun 2020 22:02:40 +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|02qps2:i</customfieldvalue>

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

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

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