Discussion :

[bisontin]

Bonjour,
voila j'ai fait une petite modif ds un test d'un code de calcul, ce test utlise une viscosité _mu et le but de ma modification est d'utiliser deux viscosités _mu1 et _mu2;
à la ligne 288 de NavierStokesSolverPC.hpp le code utlise la fonction stiff qui va la chercher dans elemOper.hpp (ligne 953) et elemOper.cpp (ligne 60), dans ces deux fichiers j'ai modifié les arguments de la fonction stiff ainsi dans NavierStokesSolverPC.hpp. quand je lance le test j'ai une erreur:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
main.o(.gnu.linkonce.t._ZN5LifeV20NavierStokesSolverPCINS_12RegionMesh3DINS_11LinearTetraENS_12MarkerCommonINS_17MarkerTraits_BaseEEEEEEC1ERK6GetPotRKNS_5RefFEESD_RKNS_8QuadRuleESG_SG_SG_RNS_9BCHandlerERKi+0xcb1): In function `LifeV::NavierStokesSolverPC<LifeV::RegionMesh3D<LifeV::LinearTetra, LifeV::MarkerCommon<LifeV::MarkerTraits_Base> > >::NavierStokesSolverPC(GetPot const&, LifeV::RefFE const&, LifeV::RefFE const&, LifeV::QuadRule const&, LifeV::QuadRule const&, LifeV::QuadRule const&, LifeV::QuadRule const&, LifeV::BCHandler&, int const&)':
: undefined reference to `LifeV::stiff(double, double, LifeV::ElemMat&, LifeV::CurrentFE const&, int, int, int)'
collect2: ld returned 1 exit status
make[1]: *** [test_ns_cyl] Erreur 1

elemOper.cpp : http://cjoint.com/?kznXnogUDJ
Avez vous une ideé sur la nature de l'erreur

[zooro]

Bonjour,

Oui, aucun prototype compatible de la fonction LifeV::stiff n'a été trouvé.
Soit la fonction n'existe pas ou n'est pas visible de l'endroit où elle est appelée, soit les arguments utilisés à l'appel ne correspondent pas aux arguments des prototypes déclarés pour la fonction.

PS: en utilisant la balise CODE lors de la rédaction de ton post, tu aurais simplifié sa lecture. Peut-être que plus de personnes auraient pris le temps de lire ton message

[bisontin]

voila j'ai pas vu la balise code
sinon je suppose que c'est les arguments utilisés à l'appel qui ne correspondent pas aux arguments déclarés pour stiff. peux tu m'eclairer un peu plus sur ce point parceque jene vois pas du tout parceque a l'appel de stiff il y a bien deux coefficients mu1 et mu2 et idem pour ceux declarés dans stiff.

[zooro]

Définition de la fonction :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
void stiff( Real coef1, Real coef2, ElemMat& elmat, const CurrentFE& fe,
            int iblock, int jblock, int nb )

Appel de la fonction avec :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
LifeV::stiff(double, double, LifeV::ElemMat&, LifeV::CurrentFE const&,
            int, int, int)

Donc déjà, il veut des "Real" et tu lui donnes des "double".

[bisontin]

Oui c'est ce que je me suis dit au début ca saute aux yuex mais avant ma modif,
c'était à l'appel :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
void stiff( Real coef, ElemMat& elmat, const CurrentFE& fe,
 int iblock, int jblock, int nb )

et ca marchait parfaitement

[zooro]

Oui c'est ce que je me suis dit au début ca saute aux yuex mais avant ma modif,
c'était à l'appel :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
void stiff( Real coef, ElemMat& elmat, const CurrentFE& fe,
 int iblock, int jblock, int nb )

et ca marchait parfaitement

Bizarre.
Deux solutions simples :
- soit tu forces le cast des double en Real (si ça marchait avant ta modif, ça doit être possible);
- soit tu ajoutes un prototype avec des doubles à la place des Real.

[bisontin]

oulla oulla,
et qu'est ce que ca veut dire tout ca , je n'ai pas de formation programmation
merci zooro

[bisontin]

en fait je voulais ajouter une information que moi me semblait ne peut pas etre la cause de mon souci: en fait il y a bcp de test dans ce code et vraisemblablement il y en a plein qui vont chercher la fonction stiff et dans cet appel stiff dépend d'un seul coef, donc moi qd j'ai compilé le code j'ai fait un make -k pour eviter qu'il ne s'arrete a cause de ca, donc au bout de ce make, le code va creer des excecutables que le test sur lequel je bosse va aller les chercher.
Question: est ce que j'aurai du modifier tt les fichiers qui appellent stiff ?
Merci

[zooro]

oulla oulla,
et qu'est ce que ca veut dire tout ca , je n'ai pas de formation programmation
merci zooro

En gros, un cast, c'est un moyen de dire au compilateur : tu vois ce double, là, ben c'est pas un double, c'est un Real ! Il faut faire attention de ne pas l'utiliser n'importe comment, mais ça peut aider à forcer la main au compilateur dans certains cas.

L'autre solution, c'est de copier la fonction que tu as modifiée/ajoutée, en remplaçant dans sa déclaration les "Real coef1" et "Real coef2" par des "double coef1" et "double coef2". Il y aura peut-être des adaptations à faire dans le code de la fonction.

[zooro]

en fait je voulais ajouter une information que moi me semblait ne peut pas etre la cause de mon souci: en fait il y a bcp de test dans ce code et vraisemblablement il y en a plein qui vont chercher la fonction stiff et dans cet appel stiff dépend d'un seul coef, donc moi qd j'ai compilé le code j'ai fait un make -k pour eviter qu'il ne s'arrete a cause de ca, donc au bout de ce make, le code va creer des excecutables que le test sur lequel je bosse va aller les chercher.
Question: est ce que j'aurai du modifier tt les fichiers qui appellent stiff ?
Merci

Tu as modifié la fonction stiff, ou tu en as créé une deuxième ?

Si tu as ajouté une fonction avec un deuxième paramètre, l'ancienne existe toujours, et il n'est pas utile de modifier les autres appels.

Par contre, si tu as modifié la fonction stiff, en ajoutant un paramètre, tu n'as pas le choix. Il n'existe plus qu'une seule fonction stiff, et tu es obligé de mettre à jour tous les appels à cette fonction.

[bisontin]

Tu as modifié la fonction stiff, ou tu en as créé une deuxième ?

Si tu as ajouté une fonction avec un deuxième paramètre, l'ancienne existe toujours, et il n'est pas utile de modifier les autres appels.

Par contre, si tu as modifié la fonction stiff, en ajoutant un paramètre, tu n'as pas le choix. Il n'existe plus qu'une seule fonction stiff, et tu es obligé de mettre à jour tous les appels à cette fonction.

au fait moi j'ai ajouté un deuxieme paramètre à la même fonction et qu'est ce que tu me conseille:

-Ajouter un deuxième parametre dans la meme fonction et faire cette modification partout dans le code ( dans les fichiers où elle est applée).

-ajouter une autre fonction.

[zooro]

au fait moi j'ai ajouté un deuxieme paramètre à la même fonction et qu'est ce que tu me conseille:
-Ajouter un deuxième parametre dans la meme fonction et faire cette modification partout dans le code ( dans les fichiers où elle est applée).
-ajouter une autre fonction.

Ajouter une nouvelle fonction est plus simple. En plus, tu n'as pas à retester tout ton code puisque l'existant n'a pas changé.

Par contre, peut-être que le deuxième paramètre est indispensable pour le calcul, et que l'ancienne fonction DOIT être remplacée, c'est à toi de décider (je n'en sais pas assez pour le faire à ta place).

[bisontin]

merci zooro ca marche maintenant

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
void stiff_visc( Real coef1, Real coef2, ElemMat& elmat, 
const CurrentFE& fe, int iblock, int jblock, int nb )

et bien sur dans les autres fichiers

ca parrait tellemnt simple mais c'est ce que fait la différence entre toi et moi (un pro dans la programmation et un pas )

bon je préfère ne pas mettre ce sujet à RESOLU parceque je vais en avoir besoin d'aide
Merci encore Zooro