<!-- 
RSS generated by JIRA (8.3.4#803005-sha1:1f96e09b3c60279a408a2ae47be3c745f571388b) at Sat Feb 10 15:31:36 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-327] [RM-8382] [parameters checker] wrong check for velocitifit in linemeasSolve</title>
                <link>https://pfspipe.ipmu.jp/jira/browse/REDMINE1D-327</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-10-04 16:26:25 by Didier Vibert. % Done: 100&lt;/font&gt;&lt;/em&gt;&lt;/p&gt;


&lt;p&gt;MR pylibamazed: &lt;a href=&quot;https://gitlab.lam.fr/CPF/cpf-redshift/-/merge_requests/558&quot; class=&quot;external-link&quot; rel=&quot;nofollow&quot;&gt;https://gitlab.lam.fr/CPF/cpf-redshift/-/merge_requests/558&lt;/a&gt;&lt;br/&gt;
MR dataset-parameters: &lt;a href=&quot;https://gitlab.lam.fr/amazed/dataset-parameters/-/merge_requests/65&quot; class=&quot;external-link&quot; rel=&quot;nofollow&quot;&gt;https://gitlab.lam.fr/amazed/dataset-parameters/-/merge_requests/65&lt;/a&gt;&lt;/p&gt;

&lt;p&gt;when linemeasSolve and fittingmethod is &quot;lbfgs&quot; the velocityfit parameter is required.&lt;br/&gt;
and the checker is issuing a warning: &lt;/p&gt;

&lt;p&gt;&amp;lt;pre&amp;gt;&lt;br/&gt;
Warning:  Unused parameter object qso LineMeasSolve velocityfit&lt;br/&gt;
&amp;lt;/pre&amp;gt;&lt;/p&gt;

&lt;p&gt;the check is wrong because @&quot;lbfgsb&quot;@ is used for the ftting method instead of @&quot;lbfgs&quot;@ :&lt;/p&gt;

&lt;p&gt;&amp;lt;pre&amp;gt;&amp;lt;code class=&quot;python&quot;&amp;gt;&lt;br/&gt;
    def _check_linemeassolve_fittingmethod_lbfgsb_velocityfit(self, object_type: str):&lt;br/&gt;
        self._check_dependant_parameter_presence(&lt;br/&gt;
            self.accessor.get_lineMeasSolve_fittingmethod(object_type) == &quot;lbfgsb&quot;,&lt;br/&gt;
            self.accessor.get_lineMeasSolve_velocityfit(object_type) is not None,&lt;br/&gt;
            error_message=f&quot;LineMeasSolve velocityfit for object &lt;/p&gt;
{object_type}&quot;,&lt;br/&gt;
            warning_message=f&quot;object {object_type}
&lt;p&gt; LineMeasSolve velocityfit&quot;&lt;br/&gt;
        )&lt;br/&gt;
&amp;lt;/code&amp;gt;&amp;lt;/pre&amp;gt;&lt;/p&gt;
</description>
                <environment></environment>
        <key id="24038">REDMINE1D-327</key>
            <summary>[RM-8382] [parameters checker] wrong check for velocitifit in linemeasSolve</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="r2j.migrate">Redmine-Jira Migtation</assignee>
                                    <reporter username="r2j.migrate">Redmine-Jira Migtation</reporter>
                        <labels>
                    </labels>
                <created>Wed, 4 Oct 2023 18:15:56 +0000</created>
                <updated>Mon, 6 Nov 2023 18:21:54 +0000</updated>
                            <resolved>Mon, 6 Nov 2023 18:21:54 +0000</resolved>
                                                                        <due></due>
                            <votes>0</votes>
                                    <watches>1</watches>
                                                                <comments>
                            <comment id="35693" author="r2j.migrate" created="Mon, 6 Nov 2023 18:19:37 +0000"  >&lt;p&gt;Comment by Didier Vibert on 2023-10-04 16:27:19:&lt;br/&gt;
added @fdufresne as watcher&lt;/p&gt;</comment>
                            <comment id="35694" author="r2j.migrate" created="Mon, 6 Nov 2023 18:19:42 +0000"  >&lt;p&gt;Comment by Fanny Dufresne on 2023-10-12 16:06:58:&lt;br/&gt;
je vois lbfgsb dans&lt;/p&gt;
&lt;ul class=&quot;alternate&quot; type=&quot;square&quot;&gt;
	&lt;li&gt;la doc parameters&lt;/li&gt;
	&lt;li&gt;le json schema&lt;/li&gt;
	&lt;li&gt;ce check&lt;/li&gt;
	&lt;li&gt;tout le python&lt;/li&gt;
&lt;/ul&gt;


&lt;p&gt;et lbfgs dans:&lt;/p&gt;
&lt;ul class=&quot;alternate&quot; type=&quot;square&quot;&gt;
	&lt;li&gt;le c++&lt;/li&gt;
&lt;/ul&gt;


&lt;p&gt;Si je comprends bien la modif &#224; faire est un petit peu plus profonde que juste le check, il faudrait modifier toutes les occurences dans le python ou bien le c ++ ? &lt;/p&gt;

&lt;p&gt;@dvibert tu peux me confirmer que c&apos;est bien &#231;a ?&lt;/p&gt;


</comment>
                            <comment id="35695" author="r2j.migrate" created="Mon, 6 Nov 2023 18:19:52 +0000"  >&lt;p&gt;Comment by Didier Vibert on 2023-10-12 17:05:46:&lt;br/&gt;
oui bien vu je confirme, &lt;/p&gt;

&lt;p&gt;la librairie externe s&apos;appelle &quot;LBFGSpp&quot; pour Limited-memory BFGS, BFGS pour Broyden&#8211;Fletcher&#8211;Goldfarb&#8211;Shanno du nom des auteurs de l&apos;algo, et pp pour C++ ! &lt;br/&gt;
Cette lib impl&#233;mente en fait 2 algos: &lt;/p&gt;
&lt;ol&gt;
	&lt;li&gt;LBFGS&lt;/li&gt;
	&lt;li&gt;LBFGSB . ce deuxi&#232;me algo, en fait assez &#233;volu&#233; par rapport au pr&#233;c&#233;dent, impl&#233;mente une contrainte de type &quot;box&quot; sur les param (d&apos;o&#249; le B final) et permet donc d&apos;imposer une contrainte min,max sur tous les param&#232;tres.&lt;/li&gt;
&lt;/ol&gt;


&lt;p&gt;c&apos;est bien LBFGSB qu&apos;on utilise, justement pour cette capacit&#233; de contrainte, donc effectivement, si tu peux renommer dans le C+&lt;ins&gt;, le lbfgs en lbfgsb welcome (mais ne modifie pas les noms provenant de la lib externe LBFGSpp). Sinon, tu peux aussi te limiter &#224; modifier le C&lt;/ins&gt;+ uniquement pour prendre &quot;lbfgsb&quot; comme valeur du param&#232;tre fittingmethod.&lt;/p&gt;</comment>
                            <comment id="35696" author="r2j.migrate" created="Mon, 6 Nov 2023 18:20:25 +0000"  >&lt;p&gt;Comment by Fanny Dufresne on 2023-10-13 07:09:16:&lt;br/&gt;
Ok super, merci pour les explications ! je vais juste remplacer les apparitions du mot &quot;lbfgs&quot; par &quot;lbfgsb&quot;&lt;/p&gt;

&lt;p&gt;C&apos;est le CLbfgsFitter en particulier que tu proposes de renommer en CLbfgsbFitter ?&lt;/p&gt;</comment>
                            <comment id="35697" author="r2j.migrate" created="Mon, 6 Nov 2023 18:21:00 +0000"  >&lt;p&gt;Comment by Didier Vibert on 2023-10-19 09:08:15:&lt;br/&gt;
il y a aussi des modifs &#224; faire sur dataset-parameters (variation linemeas_lbfgsb.json, ...)&lt;br/&gt;
peut &#234;tre faire un grep lbfgs sur tous les json parameters &lt;/p&gt;

&lt;p&gt;c&apos;est &#233;tonnant d&apos;ailleurs, les IT sont pass&#233;s  sans changer  linemeas_lbfgsb.json ? &lt;/p&gt;</comment>
                            <comment id="35698" author="r2j.migrate" created="Mon, 6 Nov 2023 18:21:33 +0000"  >&lt;p&gt;Comment by Fanny Dufresne on 2023-10-19 14:59:38:&lt;br/&gt;
C&apos;est modifi&#233; ! Il se pourrait que je n&apos;ai pas fait passer tous les ITs auparavant ^^&lt;br/&gt;
Mais maintenant ils passent&lt;/p&gt;</comment>
                            <comment id="35699" author="r2j.migrate" created="Mon, 6 Nov 2023 18:21:43 +0000"  >&lt;p&gt;Comment by Pierre-yves Chabaud on 2023-10-24 14:37:16:&lt;br/&gt;
Merged into @develop@ (@b391592c@)&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|zzsztj:</customfieldvalue>

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