[REDMINE1D-31] [RM-6196] Review operator status Created: 04/Jun/21  Updated: 05/Dec/23

Status: In Progress
Project: 1D Redmine
Component/s: None
Affects Version/s: None
Fix Version/s: None

Type: Task Priority: Normal
Reporter: Redmine-Jira Migtation Assignee: Redmine-Jira Migtation
Resolution: Unresolved Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified


 Description   

Created on 2020-12-16 10:36:48 by Ali Allaoui. % Done: 20

Operators have a status type, not always used, that refers to an error :
@
nStatus_OK = 0,
nStatus_DataError,
nStatus_LoopError,
nStatus_InvalidProductsError,
nStatus_NoOverlap
@
This should be integrated to error codes (or quality flag ?). When such status is set, should we throw an exception ?



 Comments   
Comment by Redmine-Jira Migtation [ 05/Dec/23 ]

Comment by Ali Allaoui on 2022-05-09 16:12:00:
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 ?

Comment by Redmine-Jira Migtation [ 05/Dec/23 ]

Comment by Didier Vibert on 2022-05-12 07:20:03:
Ali Allaoui wrote in #note-10:
> 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 ?

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.

Comment by Redmine-Jira Migtation [ 05/Dec/23 ]

Comment by Didier Vibert on 2022-05-12 07:22:44:
le @nStatus_InvalidProductsErrors@ n'est pas (plus ?) utilisé (testé mais jamais mis => il faut supprimer les tests)

Comment by Redmine-Jira Migtation [ 05/Dec/23 ]

Comment by Ali Allaoui on 2023-09-18 16:02:08:
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 ?
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

Generated at Sat Feb 10 15:28:35 JST 2024 using Jira 8.3.4#803005-sha1:1f96e09b3c60279a408a2ae47be3c745f571388b.