search

diyabc / diyabcGUI

User interface to DIYABC/AbcRanger
https://diyabc.github.io/
Other
4 stars 2 forks source link

[RF][param estim] update and check parameter to be estimated depending on the selected model #88

Closed gdurif closed 3 years ago

gdurif commented 3 years ago

RF estimation de paramètre: si je demande a estimer un parametre r (taux d'admixture) dans un modele (par exemple modele 1) qui n'a pas ce parametre (sachant que ce parametre est present dans un autre modele du groupe de modeles comparés - le modele 3 en l'occurence - et donc indiqué dans la liste des parametres qui est multimodele), l'interface ecrit "Parameter to estimate is ok" et les analyses RF se deroulent en apparence dans probleme (avec des resultats super louches a la fin however ce qui se comprends)...alors que ca ne devrait pas marcher du tout car ce parametre n'est pas present dans le modele 1 !!!

---> il faudrait un check que le parametre est bien present dans le modele selectionné et si c'est pas le cas un message qui dit "Parameter not present in the chosen scenario".

gdurif commented 3 years ago

Fixed in 377f760 (parameter list update) and 2a87350 (parameter check)

gdurif commented 3 years ago

Should be in next release (>1.0.13)

gdurif commented 3 years ago

-> validated but following comment:

attention les operations possibles entre parametres sont "+" , "/" et "*" et il manque dans ta phrase un exemple pour "*". Typiquement quand on a des taux de mutations (par exemple avec des microsat) on aime bien estimer le parametre composite N*µmic_1 (taille efficace taux de mutation moyen). Bref ta phrase explicative deviendrait "You can use one of the following parameter or an arithmetic combination of them, such as division, addition or multiplication of two existing parameters, for instance 't/N' or 't1+t2' or 'N\µmic_1'

gdurif commented 3 years ago

Fixed in 89c5f4b