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/REDMINE1D-31 1D Redmine <p><em><font color="#505f79"> Created on 2020-12-16 10:36:48 by Ali Allaoui. % Done: 20</font></em></p> <p>Operators have a status type, not always used, that refers to an error :<br/> @<br/> nStatus_OK = 0, <br/> nStatus_DataError, <br/> nStatus_LoopError, <br/> nStatus_InvalidProductsError, <br/> nStatus_NoOverlap <br/> @<br/> This should be integrated to error codes (or quality flag ?). When such status is set, should we throw an exception ?</p> REDMINE1D-31

[RM-6196] Review operator status

Task Normal In Progress Unresolved Redmine-Jira Migtation Redmine-Jira Migtation Fri, 4 Jun 2021 01:20:07 +0000 Mon, 4 Dec 2023 18:56:58 +0000 0 1 <p>Comment by Ali Allaoui on 2022-05-09 16:12:00:<br/> Same thing with CGaussianFit::ESStatus, should we throw an exception and move these codes (except nStatus_FailToReachTolerance) to ErrorCode ? and move nStatus_FailToReachTolerance to warning flags ?</p> <p>Comment by Didier Vibert on 2022-05-12 07:20:03:<br/> Ali Allaoui wrote in #note-10:<br/> &gt; Same thing with CGaussianFit::ESStatus, should we throw an exception and move these codes (except nStatus_FailToReachTolerance) to ErrorCode ? and move nStatus_FailToReachTolerance to warning flags ?</p> <p>je pense qu'il faut remplacer partout les status not ok par des exceptions. Je vais lister partout et voir ce qu'il en est.</p> <p>Comment by Didier Vibert on 2022-05-12 07:22:44:<br/> le @nStatus_InvalidProductsErrors@ n'est pas (plus ?) utilisé (testé mais jamais mis =&gt; il faut supprimer les tests)</p> <p>Comment by Ali Allaoui on 2023-09-18 16:02:08:<br/> Je ressuscite ce vieux ticket : aujourd'hui on throw depuis le code appelant si le status est différent de OK. Est ce qu'on ne voudrait pas plutôt lancer une exception au lieu de retourner un code ?<br/> En fait ce n'est pas toujours le cas, par exemple dans templateFitting certains codes vont être interprété comme des warnings, mais là aussi on pourrait mettre les warnings au lieux de renseigner un code d'erreur</p> Development Rank 0|zzs7ds: