<!-- 
RSS generated by JIRA (8.3.4#803005-sha1:1f96e09b3c60279a408a2ae47be3c745f571388b) at Sat Feb 10 16:38:27 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-1696] Store offset values in gen2 MEMORY status dict</title>
                <link>https://pfspipe.ipmu.jp/jira/browse/INSTRM-1696</link>
                <project id="10300" key="INSTRM">Instrument control development</project>
                    <description>&lt;p&gt;When any gen2 script makes an offset, that should be recorded in some MEMORY slots. These will be relayed into the PFS system and recorded at the start of each AG/SPS exposure. Among other things, this means that the MEMORY values should be &lt;b&gt;cleared&lt;/b&gt; when the actual offsets are cleared by slewing, etc.&lt;/p&gt;

&lt;p&gt;The &quot;dither offsets&quot; are by far the most important. &#160;Rotator and focus &quot;guide offsets&quot; are also desirable, but any other applied moves (az and alt) should be stored if possible.&lt;/p&gt;

&lt;p&gt;All keys should go into the &lt;tt&gt;MEMORY.PFS&lt;/tt&gt; table space. Under that, I think we can just use a flat namespace, with &lt;tt&gt;DITHER_RA&lt;/tt&gt; and &lt;tt&gt;DITHER_DEC&lt;/tt&gt;, etc. for the dithers, and &lt;tt&gt;AG_IR&lt;/tt&gt;, &lt;tt&gt;AG_FOCUS&lt;/tt&gt;, etc. for guide offsets.&lt;/p&gt;

&lt;p&gt;If we ever expect to make user offsets for other reasons (i.e. not just dithering), I propose we use &lt;tt&gt;USEROFFSET_RA&lt;/tt&gt;, etc. instead of &lt;tt&gt;DITHER_RA&lt;/tt&gt;.&lt;/p&gt;</description>
                <environment></environment>
        <key id="22938">INSTRM-1696</key>
            <summary>Store offset values in gen2 MEMORY status dict</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="hiro">Yoshida, Hiroshige</assignee>
                                    <reporter username="cloomis">cloomis</reporter>
                        <labels>
                            <label>EngRun</label>
                    </labels>
                <created>Tue, 13 Sep 2022 17:27:04 +0000</created>
                <updated>Tue, 11 Oct 2022 19:15:38 +0000</updated>
                            <resolved>Tue, 11 Oct 2022 19:15:38 +0000</resolved>
                                                                    <component>PFS_kansoku</component>
                        <due></due>
                            <votes>0</votes>
                                    <watches>4</watches>
                                                                <comments>
                            <comment id="31325" author="hiro" created="Tue, 13 Sep 2022 19:23:15 +0000"  >&lt;p&gt;Following the proposed design, for dithering offsets, at least the AG actor wants&lt;/p&gt;

&lt;p&gt;&lt;tt&gt;DITHER_RA&lt;/tt&gt; (arcsec) : offset to the RA coordinate, not angular separation on the tangent plane&lt;br/&gt;
&lt;tt&gt;DITHER_DEC&lt;/tt&gt; (arcsec)&lt;br/&gt;
&lt;tt&gt;DITHER_INST_PA&lt;/tt&gt; or &lt;tt&gt;DITHER_PA&lt;/tt&gt; (arcsec)&lt;/p&gt;

&lt;p&gt;For field acquisition and auto-guiding,&#160;&lt;/p&gt;

&lt;p&gt;&lt;tt&gt;AG_RA&lt;/tt&gt; (arcsec) : cumulative guide offset to the RA coordinate, not angular separation on the tangent plane&lt;br/&gt;
&lt;tt&gt;AG_DEC&lt;/tt&gt; (arcsec)&lt;br/&gt;
&lt;tt&gt;AG_INSROT&lt;/tt&gt; or &lt;tt&gt;AG_INSTROT&lt;/tt&gt; or &lt;tt&gt;AG_INR&lt;/tt&gt; (arcsec) : correction to INSROT made by offsetting INST_PA&lt;/p&gt;

&lt;p&gt;The longest key among the proposed above is &lt;tt&gt;MEMORY.PFS.DITHER_INST_PA&lt;/tt&gt;&lt;/p&gt;</comment>
                            <comment id="31329" author="eric" created="Tue, 13 Sep 2022 22:35:25 +0000"  >&lt;p&gt;So, can we fix the names?&#160; How about DITHER_PA and AG_INR instead of the alternate spellings?&lt;/p&gt;</comment>
                            <comment id="31330" author="hiro" created="Tue, 13 Sep 2022 22:50:58 +0000"  >&lt;p&gt;I am OK with your suggestion.&lt;/p&gt;</comment>
                            <comment id="31338" author="cloomis" created="Wed, 14 Sep 2022 16:43:27 +0000"  >&lt;p&gt;Agreed: &lt;/p&gt;
&lt;div class=&quot;preformatted panel&quot; style=&quot;border-width: 1px;&quot;&gt;&lt;div class=&quot;preformattedContent panelContent&quot;&gt;
&lt;pre&gt;DITHER_{RA,DEC,PA}
&lt;/pre&gt;
&lt;/div&gt;&lt;/div&gt;

&lt;p&gt;and &lt;/p&gt;
&lt;div class=&quot;preformatted panel&quot; style=&quot;border-width: 1px;&quot;&gt;&lt;div class=&quot;preformattedContent panelContent&quot;&gt;
&lt;pre&gt;AG_{RA,DEC,INR}
&lt;/pre&gt;
&lt;/div&gt;&lt;/div&gt;</comment>
                            <comment id="31341" author="hiro" created="Wed, 14 Sep 2022 17:05:42 +0000"  >&lt;p&gt;To clarify:&lt;/p&gt;

&lt;p&gt;DITHER_RA : user offset in RA &quot;on the sky&quot; (arcsec)&lt;/p&gt;

&lt;p&gt;AG_RA : cumulative value of applied RA offsets (errors) sent from the AG actor, which is a change in the RA coordinate value, without projection on to a tangent plane (arcsec)&lt;/p&gt;</comment>
                            <comment id="31357" author="cloomis" created="Thu, 15 Sep 2022 01:27:36 +0000"  >&lt;p&gt;Please use the table in &lt;a href=&quot;https://pfspipe.ipmu.jp/jira/browse/INSTRM-1702&quot; title=&quot;Relay gen2 MEMORY.PFS status to MHS/opdb&quot; class=&quot;issue-link&quot; data-issue-key=&quot;INSTRM-1702&quot;&gt;&lt;del&gt;INSTRM-1702&lt;/del&gt;&lt;/a&gt; to find the official keyword names.&lt;/p&gt;</comment>
                            <comment id="31358" author="eric" created="Thu, 15 Sep 2022 02:31:07 +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;, so I am summing -1 * offset for every ra, dec, rot offset....correct?&#160; This is easy to change if later you decide otherwise.&lt;/p&gt;</comment>
                            <comment id="31359" author="hiro" created="Thu, 15 Sep 2022 03:26:57 +0000"  >&lt;p&gt;Please give us the sum of values that are sent from the PFS side and actually applied to the telescope command position, without any scaling &lt;del&gt;or sign flipping&lt;/del&gt;. Cumulative guide offsets are not something regular observers or support scientists have to deal with.&lt;/p&gt;</comment>
                            <comment id="31362" author="eric" created="Thu, 15 Sep 2022 06:07:39 +0000"  >&lt;p&gt;To clarify: we are applying the negation of the values you are sending for guide offsets. This seemed to be working in the last engineering. BUT, you want me to sum the non-negated values, is that right?&#160; Just want to make sure...&lt;/p&gt;

&lt;p&gt;&#160;&lt;/p&gt;</comment>
                            <comment id="31369" author="hiro" created="Thu, 15 Sep 2022 18:07:39 +0000"  >&lt;p&gt;Sorry, negated values (errors -&amp;gt; offsets) are what I initially proposed. So please sum up &lt;b&gt;negated&lt;/b&gt; values that were actually applied to the telescope command position.&lt;/p&gt;</comment>
                            <comment id="31554" author="yuki.moritani" created="Fri, 7 Oct 2022 02:25:47 +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;is it OK to close this, as MEMORY statuses were stored correctly during the last run?&lt;/p&gt;</comment>
                            <comment id="31577" author="yuki.moritani" created="Tue, 11 Oct 2022 19:15:38 +0000"  >&lt;p&gt;It can be closed as the gen2 MEMORY statuses were passed to agActor as MHS statuses in the last run. (If a bug/missing item is fount, it will be addressed under a new ticket.)&lt;/p&gt;</comment>
                    </comments>
                <issuelinks>
                            <issuelinktype id="10000">
                    <name>Blocks</name>
                                            <outwardlinks description="blocks">
                                        <issuelink>
            <issuekey id="22945">INSTRM-1702</issuekey>
        </issuelink>
                            </outwardlinks>
                                                        </issuelinktype>
                            <issuelinktype id="10003">
                    <name>Relates</name>
                                            <outwardlinks description="relates to">
                                        <issuelink>
            <issuekey id="22946">INSTRM-1703</issuekey>
        </issuelink>
                            </outwardlinks>
                                                        </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|02qpio:fi</customfieldvalue>

                        </customfieldvalues>
                    </customfield>
                                                                                                                        <customfield id="customfield_10005" key="com.pyxis.greenhopper.jira:gh-sprint">
                        <customfieldname>Sprint</customfieldname>
                        <customfieldvalues>
                                <customfieldvalue id="152">preEngRun07Sep</customfieldvalue>

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