Occa: libCEED Travis CI avec OCCA échoue maintenant

Créé le 26 août 2018  ·  12Commentaires  ·  Source: libocca/occa

Une mise à jour récente d'OCCA donne maintenant un échec dans Travis pour OCCA pour tous les exemples, CEED, MFEM et PETSc. Je soupçonne qu'il y a un changement qui doit être mis à jour dans nos backends OCCA

Lien PR avec Travis défaillant :
https://github.com/CEED/libCEED/pull/143

Tous les 12 commentaires

Mon mauvais, accidentellement poussé un mauvais commit à maîtriser

Pas de soucis. Je voulais juste m'assurer que vous le saviez, car cela aurait pu être quelque chose que nous devions ajuster de notre côté.

Je vois toujours le problème. J'ai fait un PR pour libCEED qui exécute Travis avec la dernière version d'OCCA avec laquelle libCEED passe.

https://github.com/CEED/libCEED/pull/145

Oups, mal lu le hachage. Je pensais que c'était le commit HEAD à l'époque.
Je vais vérifier ça ce soir, mon mauvais

Pas de soucis, merci d'avoir regardé

@jeremylt semble que la nouvelle méthode kernelHash gâche quelque chose

https://github.com/libocca/occa/blob/master/src/device.cpp#L442

Le remettre à occa::hash(allProps) semble faire à nouveau passer les tests libCEED. Je vais continuer à chercher pour trouver la cause première, je voulais juste vous mettre à jour à ce sujet

Soit dit en passant, @dmed256 , je pense que ce problème affecte également la branche moteurs-dev de mfem et probablement aussi occa-dev.

Merci pour la mise à jour!

@jeremylt @v-dobrev Je viens de tester libCEED avec origin/master (1450f7420484e984adc57ab5df070bd03c0ccbb8) et ça passe à nouveau

Merci David ! Je vais essayer avec mfem.

Le dernier commit (https://github.com/libocca/occa/commit/780a06ae273ea037bf21452600ffdb17b93a312c) fonctionne correctement avec mfem (engines-dev).

Cette page vous a été utile?
0 / 5 - 0 notes