<!-- 
RSS generated by JIRA (8.3.4#803005-sha1:1f96e09b3c60279a408a2ae47be3c745f571388b) at Sat Feb 10 16:31:14 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>[INSTRM-1037] Store AG geometry in pfs_insdata repo</title>
                <link>https://pfspipe.ipmu.jp/jira/browse/INSTRM-1037</link>
                <project id="10300" key="INSTRM">Instrument control development</project>
                    <description>&lt;p&gt;AG cameras position on PFI is needed for &quot;shuffle&quot; to find guide stars.&#160; Following the discussions at ICS/PFI+MCS telecon on 3 and 10 July 2020, 4 corner of each sensor is proper.&lt;/p&gt;

&lt;p&gt;&#160;&lt;/p&gt;

&lt;p&gt;Hassan, Naoyuki and I talked about the file architecture of this information a while ago, looking the current&#160;&lt;tt&gt;pfi.yaml&lt;/tt&gt; and Cobra stuffs. To have commonality with Cobra, we&apos;d like to propose:&lt;/p&gt;
&lt;ul&gt;
	&lt;li&gt;List available AG cameras in&#160;&lt;tt&gt;pfi.yaml&lt;/tt&gt; (basically all 6 cameras)&lt;/li&gt;
	&lt;li&gt;Store 4 corners in another .yaml file (1 file for all cameras, or 6 files per camera)&lt;/li&gt;
&lt;/ul&gt;


&lt;p&gt;&#160;&lt;/p&gt;</description>
                <environment></environment>
        <key id="14627">INSTRM-1037</key>
            <summary>Store AG geometry in pfs_insdata repo</summary>
                <type id="3" iconUrl="https://pfspipe.ipmu.jp/jira/secure/viewavatar?size=xsmall&amp;avatarId=10518&amp;avatarType=issuetype">Task</type>
                                            <priority id="3" iconUrl="https://pfspipe.ipmu.jp/jira/images/icons/priorities/major.svg">Major</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="hiro">Yoshida, Hiroshige</assignee>
                                    <reporter username="yuki.moritani">yuki.moritani</reporter>
                        <labels>
                            <label>EngRun</label>
                    </labels>
                <created>Fri, 10 Jul 2020 14:38:28 +0000</created>
                <updated>Fri, 17 Dec 2021 12:35:40 +0000</updated>
                            <resolved>Mon, 13 Sep 2021 19:11:12 +0000</resolved>
                                                                    <component>pfs_instdata</component>
                        <due></due>
                            <votes>0</votes>
                                    <watches>5</watches>
                                                                <comments>
                            <comment id="19088" author="martin.reinecke" created="Fri, 2 Apr 2021 13:43:47 +0000"  >&lt;p&gt;Since shuffle is required to output pixel coordinates for guide star candidates, the list of four corners is no longer sufficient. To determine this information, we also need to know the &quot;origin&quot; corner, the &quot;+X&quot; and &quot;+Y&quot; corner, and the camera resolution.&lt;/p&gt;</comment>
                            <comment id="19089" author="martin.reinecke" created="Fri, 2 Apr 2021 13:51:53 +0000"  >&lt;p&gt;Alternatively, a per-camera matrix transforming between PFI and the respective pixel coordinate system shold also work well. Then the only additional required information would be the number of pixels in x and y direction.&lt;/p&gt;</comment>
                            <comment id="19094" author="yuki.moritani" created="Sun, 4 Apr 2021 23:37:17 +0000"  >&lt;p&gt;&lt;a href=&quot;https://pfspipe.ipmu.jp/jira/secure/ViewProfile.jspa?name=hiro&quot; class=&quot;user-hover&quot; rel=&quot;hiro&quot;&gt;Yoshida, Hiroshige&lt;/a&gt;&#160;how do you treat PFI&lt;span class=&quot;error&quot;&gt;&amp;#91;mm&amp;#93;&lt;/span&gt;-AG&lt;span class=&quot;error&quot;&gt;&amp;#91;pixel&amp;#93;&lt;/span&gt; conversion for agActor (and other AG software)? What information do you use?&lt;/p&gt;</comment>
                            <comment id="19095" author="hiro" created="Mon, 5 Apr 2021 02:58:25 +0000"  >&lt;p&gt;Currently, the AG subsystem implements naive transformation methods internally, using:&lt;/p&gt;
&lt;ul&gt;
	&lt;li&gt;reference position of the detectors (center: x = y = 511.5 pix)&lt;/li&gt;
	&lt;li&gt;PFI coordinates of the reference position (r = 241.292 mm, PA = 0, 60, 120, ... deg)&lt;/li&gt;
	&lt;li&gt;rotation angle of the detectors about the reference position (theta = 0, 60, 120, ... deg)&lt;/li&gt;
	&lt;li&gt;detector pixel pitch (p = 13 um)&lt;/li&gt;
&lt;/ul&gt;


&lt;p&gt;In general, ignoring distortion, I think we only need, for each AGC:&lt;/p&gt;
&lt;ul&gt;
	&lt;li&gt;reference position of the detector (x, y) pix&lt;/li&gt;
	&lt;li&gt;PFI coordinates of the reference position (x, y) mm&lt;/li&gt;
	&lt;li&gt;rotation angle of the detector in PFI coordinates (theta,) deg or rad&lt;/li&gt;
	&lt;li&gt;detector pitch (x, y) um&lt;/li&gt;
&lt;/ul&gt;


&lt;p&gt;In stead (or in addition), we could have transformation matrices for each AGC for each direction.&lt;/p&gt;

&lt;p&gt;&#160;&lt;/p&gt;</comment>
                            <comment id="19265" author="yuki.moritani" created="Tue, 4 May 2021 23:48:18 +0000"  >&lt;p&gt;Yoshida-san and I talked about this topic. We think it will be nice to have a module to query the PFI position for a AG cameras, and vice versa. In this way, MPA can get PFI coordinates of 4 corners of individual AG cameras, and NAOJ can can get PFI coordinates of observed centroids.. Yoshida-san already has something to do so, so I think his routine should be a good start.&lt;/p&gt;</comment>
                            <comment id="21737" author="hassan" created="Fri, 23 Jul 2021 14:07:06 +0000"  >&lt;p&gt;&lt;a href=&quot;https://pfspipe.ipmu.jp/jira/secure/ViewProfile.jspa?name=Martin.Reinecke&quot; class=&quot;user-hover&quot; rel=&quot;Martin.Reinecke&quot;&gt;Martin Reinecke&lt;/a&gt; mentioned during the ICS/PFI telecon today that he needs this in order to provide shuffle-related deliveries.&lt;/p&gt;</comment>
                            <comment id="21945" author="yuki.moritani" created="Mon, 13 Sep 2021 19:11:12 +0000"  >&lt;p&gt;The function was implemented, and Martin is now using this function, so I close this ticket.&lt;/p&gt;

&lt;p&gt;What we may need to modify in the future is to treat the fact that AG pixel coordinates includes overscan region.&lt;/p&gt;</comment>
                    </comments>
                <issuelinks>
                            <issuelinktype id="10003">
                    <name>Relates</name>
                                            <outwardlinks description="relates to">
                                        <issuelink>
            <issuekey id="11965">INFRA-207</issuekey>
        </issuelink>
                            </outwardlinks>
                                                                <inwardlinks description="relates to">
                                        <issuelink>
            <issuekey id="14623">FIBERALLOC-29</issuekey>
        </issuelink>
                            </inwardlinks>
                                    </issuelinktype>
                    </issuelinks>
                <attachments>
                    </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|02qpqj:c</customfieldvalue>

                        </customfieldvalues>
                    </customfield>
                                                                                                                                                    <customfield id="customfield_10002" key="com.atlassian.jira.plugin.system.customfieldtypes:float">
                        <customfieldname>Story Points</customfieldname>
                        <customfieldvalues>
                            <customfieldvalue>2.0</customfieldvalue>
                        </customfieldvalues>
                    </customfield>
                                                                                                                        </customfields>
    </item>
</channel>
</rss>