<!-- 
RSS generated by JIRA (8.3.4#803005-sha1:1f96e09b3c60279a408a2ae47be3c745f571388b) at Sat Feb 10 16:52:45 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>[FIBERALLOC-3] Objects in a distance &gt;190mm are discarded</title>
                <link>https://pfspipe.ipmu.jp/jira/browse/FIBERALLOC-3</link>
                <project id="10500" key="FIBERALLOC">Target to fiber allocation and configuration</project>
                    <description>&lt;p&gt;I understand this is for a testing purpose, but please remove this constraint or loosen (say 250 mm?). Now the distribution of allocated objects looks like &quot;circle&quot; not &quot;hexagon&quot;.&lt;/p&gt;</description>
                <environment></environment>
        <key id="11308">FIBERALLOC-3</key>
            <summary>Objects in a distance &gt;190mm are discarded</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="Martin.Reinecke">Martin Reinecke</assignee>
                                    <reporter username="kiyoto.yabe">Kiyoto Yabe</reporter>
                        <labels>
                    </labels>
                <created>Wed, 9 Nov 2016 05:15:51 +0000</created>
                <updated>Wed, 2 May 2018 06:15:37 +0000</updated>
                            <resolved>Wed, 2 May 2018 06:15:37 +0000</resolved>
                                                                        <due></due>
                            <votes>0</votes>
                                    <watches>3</watches>
                                                                <comments>
                            <comment id="11518" author="martin.reinecke" created="Wed, 9 Nov 2016 11:24:26 +0000"  >&lt;p&gt;I can certainly remove this radius limitation. The only problem (as long as the code is being tested for correctness and quality) is that, if we do this, the &quot;fract&quot; parameter immediately loses its meaning, since the (unconstrained) input data set will most likely contain targets that the telescope can never observe in the requested configuration. So it will be hard to experiment with questions like &quot;how many exposures do we need for observing at least x percent of the targets with a particular allocation strategy?&quot;&lt;/p&gt;

&lt;p&gt;A possible compromise would be to first determine a lilst of all targets that the telescope can observe in all its allowed positions (nposang*nptg*nptg), and use this to determine the fractions. However that will artificially increase the required number of exposures, since some of the targets will only be visible for a single telescope position.&lt;/p&gt;

&lt;p&gt;Alternatively I can change the code in a way that it doesn&apos;t constrain the target list, and instead of a fraction of sources to observe, it just takes a maximum number of exposures to perform.&lt;/p&gt;

&lt;p&gt;I&apos;d appreciate any comment on this.&lt;/p&gt;</comment>
                            <comment id="11522" author="martin.reinecke" created="Wed, 9 Nov 2016 13:04:17 +0000"  >&lt;p&gt;I have an implementation ready which removes the radius constraint and replaces the &quot;fract&quot; parameter with an &quot;n_exposures&quot; parameter. I&apos;ll commit if we conclude that this is the desired solution.&lt;/p&gt;</comment>
                            <comment id="11527" author="kiyoto.yabe" created="Thu, 10 Nov 2016 07:29:50 +0000"  >&lt;p&gt;Yes, I got your concern. Probably putting constraint is OK for calculation of the fraction, but the current value of 190mm (probably the width of the FoV in X direction) is too small for the hexagonal FoV. I propose 230 mm (the width in Y direction), so that we can define the circumscribed circle as the available region.&lt;/p&gt;</comment>
                            <comment id="11530" author="martin.reinecke" created="Thu, 10 Nov 2016 11:26:23 +0000"  >&lt;p&gt;If we choose 230mm as the cutoff radius, then the PFS will still not be able to observe all positions in that circle, unless we allow completely free rotation. And even if we do, the probability of a target being observable drops very quickly in this outer region. (E.g. a target that&apos;s at a distance of 230mm from  the center will  only be observable by choosing a very specific PA, whereas a target at a distance of 190mm will be observable by almost any PA.)&lt;br/&gt;
Using this large radius and specifying a large fraction of targets to observe will result in very, very many exposures which, at the end, are trying to observe single targets at the largest radii. This is not a realistic situation (I hope).&lt;br/&gt;
It is very possible that I am misunderstanding your suggestion. If so, please let us discuss further.&lt;/p&gt;</comment>
                            <comment id="11535" author="kiyoto.yabe" created="Fri, 11 Nov 2016 08:10:33 +0000"  >&lt;p&gt;I think it is true that the probability of observation is low in outer region, and the &quot;fract&quot; parameter may never reach the required value (especially if we fix the position angle), but I think it seems to be OK. We should stops the code if the change of the fraction almost converges (the current code does so, I think). I think the only problem in the current situation is that we can never assign objects within the FoV but outside the 190mm circle. Perhaps, I am wrong on my side...&lt;/p&gt;</comment>
                            <comment id="11539" author="martin.reinecke" created="Fri, 11 Nov 2016 12:36:10 +0000"  >&lt;p&gt;As I said, these problems can be overcome for the most part if we first establish a list of targets that can be observed by any of the possible telescope configurations and then run the code for a predefined number of observations.&lt;br/&gt;
I have the necessary changes in my local tree, and they behave exactly as expected. I will commit them to the repository for evaluation. It&apos;s easy to revert them if desired.&lt;/p&gt;</comment>
                            <comment id="11548" author="kiyoto.yabe" created="Mon, 14 Nov 2016 02:37:49 +0000"  >&lt;p&gt;Thank you for pushing you updates. I will check this version.&lt;/p&gt;</comment>
                            <comment id="13328" author="rhl" created="Tue, 1 May 2018 13:07:33 +0000"  >&lt;p&gt;Where are these numbers coming from? &#160;We need &lt;b&gt;one&lt;/b&gt; source of PFS numbers that everyone can import. &#160;A possible place would be in the datamodel project on GitHub, but we could invent another. &#160;The mappings from cobra-&amp;gt;slit should be there too.&lt;/p&gt;

&lt;p&gt;&#160;&lt;/p&gt;</comment>
                            <comment id="13329" author="rhl" created="Tue, 1 May 2018 13:08:47 +0000"  >&lt;p&gt;And I&apos;m totally confused by the discussion! &#160;Why do we need a field-radius when we know the exact geometry of the cobras? &#160;Is there some approximation being made somewhere that affects the targeting (and if so does it affect the network-flow solver?)&lt;/p&gt;</comment>
                            <comment id="13340" author="martin.reinecke" created="Tue, 1 May 2018 13:57:50 +0000"  >&lt;p&gt;Hi Robert,&lt;/p&gt;

&lt;p&gt;no need to worry, I think. This issue is actually closed since late 2016, and I&apos;m not sure why it was reopened (along with a few others) a few minutes ago. The current software does not have any such constraints.&lt;/p&gt;</comment>
                    </comments>
                    <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|ii00w7:</customfieldvalue>

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