<!-- 
RSS generated by JIRA (8.3.4#803005-sha1:1f96e09b3c60279a408a2ae47be3c745f571388b) at Sat Feb 10 16:28:24 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-763] Changes to schema for CobraConfig and mcsData</title>
                <link>https://pfspipe.ipmu.jp/jira/browse/INSTRM-763</link>
                <project id="10300" key="INSTRM">Instrument control development</project>
                    <description>&lt;p&gt;When using CobraConfig it&apos;s usually necessary to join to mcsData purely to get the frameId.  We should probably denormalise a little further (i.e. add frameId or maybe visitId), and take the opportunity to use the same name in the two tables (&lt;tt&gt;centroidx&lt;/tt&gt; v. &lt;tt&gt;mcsCenter_x&lt;/tt&gt;).&lt;/p&gt;

&lt;p&gt;We might want to remove the mcs pixel coordinates from CobraConfig all together;  the two tables are used for different things, and can be easily joined when investigating e.g. camera distortions.  &lt;/p&gt;

&lt;p&gt;While renaming columns it&apos;s worth thinking about using a Hungarian-style naming convention (e.g. adding including Pix in all pixel coordinates, and/or MM for all focal plane coordinates), especially when the two coordinate systems both appear in one table.&lt;/p&gt;

&lt;p&gt;Currently both tables include information about moves (one via &lt;tt&gt;moveId&lt;/tt&gt; and one via &lt;tt&gt;iteration&lt;/tt&gt;.  We should reconsider this too;  for example, if mcsData included the nominal position in MCS pixel coordinates we might well use it for convergence analysis rather than CobraConfig;  I doubt if we would use both&lt;/p&gt;</description>
                <environment></environment>
        <key id="13753">INSTRM-763</key>
            <summary>Changes to schema for CobraConfig and mcsData</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="kiyoto.yabe">Kiyoto Yabe</assignee>
                                    <reporter username="rhl">rhl</reporter>
                        <labels>
                    </labels>
                <created>Sat, 7 Sep 2019 15:44:45 +0000</created>
                <updated>Tue, 24 Dec 2019 01:18:37 +0000</updated>
                            <resolved>Tue, 24 Dec 2019 01:18:37 +0000</resolved>
                                                                    <component>spt_operational_database</component>
                        <due></due>
                            <votes>0</votes>
                                    <watches>3</watches>
                                                                <comments>
                            <comment id="16236" author="hassan" created="Mon, 21 Oct 2019 20:05:17 +0000"  >&lt;p&gt;I propose merging the &lt;tt&gt;cobra_config&lt;/tt&gt; and &lt;tt&gt;mcs_data&lt;/tt&gt; tables. It&apos;s not clear to me why the latter is separate. &lt;/p&gt;</comment>
                            <comment id="16239" author="kiyoto.yabe" created="Wed, 23 Oct 2019 00:50:19 +0000"  >&lt;p&gt;Historically, `mcsData` (currently `mcs_data`) was a table just containing an output of mcs_actor for only mcs measurement part, and `CobraConfig` (currently `cobra_config`) was motivated to be a table for only cobra movement part.&#160;&lt;/p&gt;

&lt;p&gt;In terms of the database normalization, I think we can merge these two because they have one-to-one correspondence, but I think we should separate two because some of the records in `cobra_config` such as pfi_center will be stored in a different process from the mcs measurement part.&#160;&lt;/p&gt;

&lt;p&gt;For your reference, you can find the current schema from here:&#160;&lt;a href=&quot;https://github.com/Subaru-PFS/spt_operational_database/blob/new_schema_oct2019/PFS-DAT-IPM003002-01_pfs_schema.pdf&quot; class=&quot;external-link&quot; rel=&quot;nofollow&quot;&gt;https://github.com/Subaru-PFS/spt_operational_database/blob/new_schema_oct2019/PFS-DAT-IPM003002-01_pfs_schema.pdf&lt;/a&gt;&lt;/p&gt;

&lt;p&gt;&#160;&lt;/p&gt;</comment>
                            <comment id="16250" author="rhl" created="Fri, 25 Oct 2019 02:13:52 +0000"  >&lt;p&gt;I agree with &lt;a href=&quot;https://pfspipe.ipmu.jp/jira/secure/ViewProfile.jspa?name=kiyoto.yabe&quot; class=&quot;user-hover&quot; rel=&quot;kiyoto.yabe&quot;&gt;Kiyoto Yabe&lt;/a&gt;;  the information in the two tables is available at at different times.&lt;/p&gt;</comment>
                            <comment id="16253" author="hassan" created="Fri, 25 Oct 2019 12:36:48 +0000"  >&lt;p&gt;Ok, I see now - if the two tables are populated in two separate processes, the tables should remain separate.&lt;/p&gt;</comment>
                    </comments>
                <issuelinks>
                            <issuelinktype id="10003">
                    <name>Relates</name>
                                                                <inwardlinks description="relates to">
                                        <issuelink>
            <issuekey id="14117">INSTRM-844</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|zy00b4:</customfieldvalue>

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