JuliaExtremes / IDFCurves.jl

MIT License
0 stars 0 forks source link

Macro refactorisation #31

Closed AugustePaoli99 closed 6 months ago

AugustePaoli99 commented 6 months ago

In the folder Structures, I would change the names of the 4 sub-folders :

The folder CorrelationStructure would contain 3 files, namely :

The folder MarginalScalingModel would contain 3 files, namely :

The folder DependentScalingModel would contain 1 file, namely dependentscalingmodel.jl

The folder EllipticalCopula would contain 3 files, namely :

jojal5 commented 6 months ago

I agree !

AugustePaoli99 commented 6 months ago

@jojal5 et pour ce qui est de changer dGEV pour GeneralScaling ?

jojal5 commented 6 months ago

Je suis d'accord aussi.

AugustePaoli99 commented 6 months ago

@jojal5 J'ai push dans la branche liée puis merge dans dev. Pour le moment les deux objets qui héritent de la classe abstraite MarginalScalingModel se nomment SimpleScaling et GeneralScaling respectivement. Mais j'ai un doute, serait-ce mieux s'ils s'appelaient SimpleScalingModel et GeneralScalingModel par exemple ?

jojal5 commented 6 months ago

Je pense que c'est ok, je n'ai pas vu de guidelines spécifiquement sur la nomenclature des types concrets. Je sais cependant que les abstract types devraient être spécifiques, par exemple AbstractScalingModel plutôt que Scaling.