https://cta-redmine.irap.omp.eu/https://cta-redmine.irap.omp.eu/favicon.ico?14312453732015-10-09T09:03:27ZCTA IRAP Project Gatewayctools - Change request #1547: ctbkgcube should set parameter boundaries of output modelhttps://cta-redmine.irap.omp.eu/issues/1547?journal_id=63462015-10-09T09:03:27ZKnödlseder Jürgenjurgen.knodlseder@irap.omp.eu
<ul></ul><p>Agree to constrain the parameters. Note that there is <code>range</code> method that allows to set both boundaries at once, i.e.<br /><pre>
spectral["Prefactor"].range(0.01, 100.0);
</pre>We might also constrain the index to some plausible values, for example [-1.0, 1.0], but we should probably check first that this does not create any problem.</p> ctools - Change request #1547: ctbkgcube should set parameter boundaries of output modelhttps://cta-redmine.irap.omp.eu/issues/1547?journal_id=63472015-10-09T09:10:12ZMayer Michaelmichael.mayer@physik.hu-berlin.de
<ul></ul><p>Even better to use the <code>range</code> method. The index is currently constrained to [-10,10] which of course is a large range. I agree that further constraining might be good and I guess [-1,1] should be fine here. To be careful we could however think about [-5,5]?</p> ctools - Change request #1547: ctbkgcube should set parameter boundaries of output modelhttps://cta-redmine.irap.omp.eu/issues/1547?journal_id=63482015-10-09T09:11:33ZKnödlseder Jürgenjurgen.knodlseder@irap.omp.eu
<ul></ul><p>Also fine. I was not aware that the index was already constrained. Whatever fit your needs should be fine, we can always revise this later when we have more experience with it.</p> ctools - Change request #1547: ctbkgcube should set parameter boundaries of output modelhttps://cta-redmine.irap.omp.eu/issues/1547?journal_id=63502015-10-09T09:20:34ZMayer Michaelmichael.mayer@physik.hu-berlin.de
<ul></ul><p>I guess in order not to break someones analysis, we could use [-5.0,5.0]. This range should still be appropriate to fit indices around 0.0.</p>
<blockquote>
<p>I was not aware that the index was already constrained.</p>
</blockquote>
<p>This happens on construction in <code>GModelSpectralPlaw::init_members()</code>.</p> ctools - Change request #1547: ctbkgcube should set parameter boundaries of output modelhttps://cta-redmine.irap.omp.eu/issues/1547?journal_id=63512015-10-09T09:27:36ZKnödlseder Jürgenjurgen.knodlseder@irap.omp.eu
<ul></ul><p>Mayer Michael wrote:</p>
<blockquote>
<p>I guess in order not to break someones analysis, we could use [-5.0,5.0]. This range should still be appropriate to fit indices around 0.0.</p>
<blockquote>
<p>I was not aware that the index was already constrained.</p>
</blockquote>
<p>This happens on construction in <code>GModelSpectralPlaw::init_members()</code>.</p>
</blockquote>
<p><img src="/plugin_assets/redmine_wiki_extensions/images/biggrin.png" alt=":D"> should remember better what I have coded ...</p> ctools - Change request #1547: ctbkgcube should set parameter boundaries of output modelhttps://cta-redmine.irap.omp.eu/issues/1547?journal_id=63522015-10-09T14:49:15ZMayer Michaelmichael.mayer@physik.hu-berlin.de
<ul></ul><p><img src="/plugin_assets/redmine_wiki_extensions/images/smile.png" alt=":)"> <br />Should I create a branch, or do you implement this right away?</p> ctools - Change request #1547: ctbkgcube should set parameter boundaries of output modelhttps://cta-redmine.irap.omp.eu/issues/1547?journal_id=63542015-10-12T13:11:30ZKnödlseder Jürgenjurgen.knodlseder@irap.omp.eu
<ul><li><strong>Status</strong> changed from <i>New</i> to <i>Closed</i></li><li><strong>Assigned To</strong> set to <i>Knödlseder Jürgen</i></li><li><strong>Target version</strong> set to <i>1.0.0</i></li><li><strong>% Done</strong> changed from <i>0</i> to <i>100</i></li></ul><p>Just implemented the change (is in <code>devel</code>).</p>