https://pfspipe.ipmu.jp/jira This file is an XML representation of an issue en-us 8.3.4 803005 13-09-2019 https://pfspipe.ipmu.jp/jira/browse/INSTRM-1841 Instrument control development <p>As reported on slack:</p> <p><a href="https://sumire-pfs.slack.com/archives/C9TDPMVHT/p1671252144208469" class="external-link" rel="nofollow">https://sumire-pfs.slack.com/archives/C9TDPMVHT/p1671252144208469</a></p> <p> A number of targets in <tt>pfsDesign</tt> <tt>0x031107ae86f9db65</tt> have unrealistic PFI positions, in that for each target the distance from the cobra center is greater than the <tt>L1+L2</tt> arm lengths.</p> <p>To avoid such situations happening in the future, I propose introducing a <tt>pfs_utils</tt> utility that validates the contents of a given <tt>pfsDesign</tt>. This utility would include at least a check for the PFI target positions.</p> INSTRM-1841

Introduce pfsDesign/Config validation code

Task Normal Done Done Kiyoto Yabe hassan EngRun Fri, 13 Jan 2023 19:43:44 +0000 Sat, 8 Jul 2023 20:10:47 +0000 Sat, 8 Jul 2023 20:10:47 +0000 0 3 <p>We talked about this topic at ICS/PFI telecon on 26 May 2023.</p> <p>Kiyoto (<a href="https://pfspipe.ipmu.jp/jira/secure/ViewProfile.jspa?name=kiyoto.yabe" class="user-hover" rel="kiyoto.yabe">Kiyoto Yabe</a>) will look into how to implement validation when pfsDesign is made. Chi-Hung (<a href="https://pfspipe.ipmu.jp/jira/secure/ViewProfile.jspa?name=chyan" class="user-hover" rel="chyan">chyan</a>) has the scripts to check design, and share with Kiyoto.</p> <p>The subaru_fiber_allocation code now checks and provides WARNING if the distance exceeds L1+L2 arm length. I can replace that part if <a href="https://pfspipe.ipmu.jp/jira/secure/ViewProfile.jspa?name=chyan" class="user-hover" rel="chyan">chyan</a> provides a better code.</p> <p>It didn't detect such case in the design file for visit=93325 (if the code works properly). If the geometry in opDB is old, probably we need to think about <a href="https://pfspipe.ipmu.jp/jira/browse/INSTRM-1644" title="Provide PFI XML file loader to cobra_geometry tables" class="issue-link" data-issue-key="INSTRM-1644"><del>INSTRM-1644</del></a>. </p> <p>In any case, this may be useful, so I'll merge the change and close this ticket.</p> Development Rank 0|zzsx5r:9