Comment by Fanny Dufresne on 2023-10-11 09:24:47:
Je propose de séparer ce ticket en 2 étapes :
1. Refacto de objects : passage de

• <object_type>: {...}

  à:

• solvers : [ {name: <object_type>, ...}

  , ...]

2. Refacto des solvers : ajouter de la description des stages - besoin de quelques clarifications là dessus

Comment by Didier Vibert on 2023-10-11 09:32:10:
Fanny Dufresne wrote in #note-6:
> Je propose de séparer ce ticket en 2 étapes :
> 1. Refacto de objects : passage de
> * <object_type>:

> à:
> * solvers : [

, ...]
>
> 2. Refacto des solvers : ajouter de la description des stages - besoin de quelques clarifications là dessus

ok p moi

Comment by Didier Vibert on 2023-10-13 14:09:16:
je mets ici un point du checker à corriger (peut-être une issue dédiée, à toi de voir @fdufresne)

dans le json shema, @"additionalProperties": false@ n'est appliqué qu'au niveau où il est mis, mais ne s'applique pas aux niveaux en dessous: il faudrait donc le répéter dans chaque section properties... à moins qu'il y ait un moyen de le définir globalement pour toutes les properties ?

Comment by Didier Vibert on 2023-10-13 15:02:52:
encore un truc, avec la nouvelle structure définie ici (https://projets.lam.fr/projects/amazed/wiki/2023-07-24), on rajoute un niveau (stages) à l'arbre des scopes.

On peut soit supprimer ce niveau lors de la conversion du json en input vers le json stocké dans la classe parameters et passé au C++, mais c'est peut-être dommage voire impossible pour l'utilisation des paramètres en python.

une solution pour facilement prendre en compte ce niveau sans trop modifier le C++ consiste à binder en python via swig la classe @CAutoscope@ et de s'en servir dans le run @Context.run@ pour pousser des noms dans la chaine scope (et les dépiler automatiquement à la destruction de l'objet). D'ailleurs dans la classe CAutoscope, on pourrait utiliser directement le singleton CProcessFlowContext pour récupérer la scopestack plutôt que de le prendre comme argument...

Autre solution définir en python un contextmanager qui push le nom en entrée et le pop en sortie, et l'utiliser avec des with statement dans le @Context.run@.

Ensuite, avec ça on peut donc pousser des scope dans la scopstack du contexte, et ça devrait donc être l'api python dans @Context.run@ qui pousse les scopes SpectrumModel et stage, => il faut modifier le constructeur C++ des methodSolve pour qu'ils ne poussent que le nom de leur méthode, et plus besoin de leur passer en argument l'object type. Celui-ci pourra être récupéré directement dans le scopeStore.

=> attention aussi, ce niveau supplémentaire dans l'arbre, va se répercuter dans le resultStore. il faudra modifier certainement un peu le code python pour en tenir compte

Comment by Fanny Dufresne on 2023-10-17 09:31:05:
Je suis tout à fait partante pour rajouter les stages à l'arbre des scopes. Par contre je n'ai pas compris grand chose à l'implémentation proposée, je passerai en discuter

Comment by Fanny Dufresne on 2023-10-18 09:27:23:
A priori les pattern properties marchent très bien, y compris au niveau de la doc. Je suis très partante pour les utilisera

Comment by Pierre-yves Chabaud on 2024-01-23 10:14:06:
Merged @develop@ (@d7a0cecc@)