<!-- 
RSS generated by JIRA (8.3.4#803005-sha1:1f96e09b3c60279a408a2ae47be3c745f571388b) at Sat Feb 10 15:29:22 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>[REDMINE1D-106] [RM-8005] [linemodel] do not switch to nocontinuum or fromspectrum in second pass continuum refit </title>
                <link>https://pfspipe.ipmu.jp/jira/browse/REDMINE1D-106</link>
                <project id="11002" key="REDMINE1D">1D Redmine </project>
                    <description>&lt;p&gt;&lt;em&gt;&lt;font color=&quot;#505f79&quot;&gt; Created on 2023-04-06 09:14:09 by Didier Vibert. % Done: 50&lt;/font&gt;&lt;/em&gt;&lt;/p&gt;


&lt;p&gt;when the continuum is refitted, eventually partially in the case @ LineModelSolve.linemodel.secondpass.continuumfit = &quot;refitfirstpass&quot;@, &lt;br/&gt;
the amplitude can be found compatible with zero (resp. negative) in the considered second pass redshift window, whereas it was found in the 1st pass full z range, that at some other redshift the continuum amplitude is significantly not null (resp. positive). Currently, the code switch to &quot;nocontinuum&quot; (resp. &quot;fromspectrum&quot;) globally , for all 2nd pass candidates.&lt;/p&gt;

&lt;p&gt;We should either make the switch for only the candidate where it appears, or not switch at all, since in first pass the continuum was found significant.&lt;br/&gt;
I vote for 2nd option. Moreover the whole pdf, outside the 2nd pass windows, was computed using continuum template fitting, thus we should keep this info and all the associated parameters (name of template, amplitude, ism, igm)&lt;/p&gt;</description>
                <environment></environment>
        <key id="23491">REDMINE1D-106</key>
            <summary>[RM-8005] [linemodel] do not switch to nocontinuum or fromspectrum in second pass continuum refit </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="3" iconUrl="https://pfspipe.ipmu.jp/jira/images/icons/statuses/inprogress.png" description="This issue is being actively worked on at the moment by the assignee.">In Progress</status>
                    <statusCategory id="4" key="indeterminate" colorName="yellow"/>
                                    <resolution id="-1">Unresolved</resolution>
                                        <assignee username="r2j.migrate">Redmine-Jira Migtation</assignee>
                                    <reporter username="r2j.migrate">Redmine-Jira Migtation</reporter>
                        <labels>
                    </labels>
                <created>Tue, 13 Jun 2023 05:00:55 +0000</created>
                <updated>Wed, 7 Feb 2024 18:23:25 +0000</updated>
                                                                                <due></due>
                            <votes>0</votes>
                                    <watches>2</watches>
                                                                <comments>
                            <comment id="37116" author="r2j.migrate" created="Wed, 7 Feb 2024 18:22:37 +0000"  >&lt;p&gt;Comment by Vincent Le Brun on 2023-04-06 10:54:17:&lt;br/&gt;
OK if the 2nd pass amplitude is found to be 0, but if it is negative, and we do not switch, an error will be raised ?&lt;/p&gt;</comment>
                            <comment id="37117" author="r2j.migrate" created="Wed, 7 Feb 2024 18:22:48 +0000"  >&lt;p&gt;Comment by Didier Vibert on 2023-04-06 12:27:02:&lt;br/&gt;
Vincent Le Brun wrote in #note-2:&lt;br/&gt;
&amp;gt; OK if the 2nd pass amplitude is found to be 0, but if it is negative, and we do not switch, an error will be raised ?&lt;/p&gt;

&lt;p&gt;The option 2 is to not raise any error which abort the object (QSO/Galaxy) fittting, but issue a warning and let the code continue with a null continuum (when it is negative, above the negative threshold it is considered null). Keep in mind that in the first pass, around the same redshifts (on a coarser grid) it was not considered negative, so we are in the case were it is at the limit of the threshold, It will probably not change anything at the end. And to introduce a &quot;fromspectrum&quot; continuum switch that is not global needs some refactoring. &lt;/p&gt;</comment>
                            <comment id="37118" author="r2j.migrate" created="Wed, 7 Feb 2024 18:22:54 +0000"  >&lt;p&gt;Comment by Didier Vibert on 2023-04-06 12:39:46:&lt;br/&gt;
Another comment: in first pass, the switch to &quot;fromspectrum&quot; arises if &lt;b&gt;at all z in the full z range&lt;/b&gt; the continuum is fitted with a negative amplitude (below the threshold). What happens currently in 2nd pass is to switch if it is fitted negative on all the z-range of one 2nd pass window. Since the 2nd pass window contains some of the z of the first pass grid, the only thing I see that can change the behavior is the use of fft in first-pass only (it is deactivated in 2nd pass because the z-range is too small to have a cpu tiime benefit).&lt;/p&gt;</comment>
                            <comment id="37119" author="r2j.migrate" created="Wed, 7 Feb 2024 18:23:03 +0000"  >&lt;p&gt;Comment by Vincent Le Brun on 2023-04-06 13:29:32:&lt;br/&gt;
OK let&apos;s go like this&lt;/p&gt;</comment>
                            <comment id="37120" author="r2j.migrate" created="Wed, 7 Feb 2024 18:23:10 +0000"  >&lt;p&gt;Comment by Gaelle Daste on 2024-01-29 12:42:40:&lt;br/&gt;
J&apos;ai pas trouv&#233; dans les ITs (m&#234;me full) de cas o&#249; on avait ce switch. Pour voir et valider le avant/apr&#232;s si quelqu&apos;un a une id&#233;e je suis preneuse&lt;/p&gt;</comment>
                            <comment id="37121" author="r2j.migrate" created="Wed, 7 Feb 2024 18:23:18 +0000"  >&lt;p&gt;Comment by Gaelle Daste on 2024-02-06 15:00:09:&lt;br/&gt;
#8580 &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|zzsxgv:</customfieldvalue>

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