lehrfempp
lehrfempp copied to clipboard
craffael
Sphere project
@ncograf Ich habe den Code noch etwas angepasst, so dass clang-tidy nicht mehr meckert. Aber ich habe jetzt grad gemerkt, dass die unit tests z.T. fehlschlagen: https://github.com/craffael/lehrfempp/actions/runs/7586593522/job/20665126047#step:7:1381
Weisst du an was das liegt?
@ncograf Ich habe den Code noch etwas angepasst, so dass clang-tidy nicht mehr meckert. Aber ich habe jetzt grad gemerkt, dass die unit tests z.T. fehlschlagen: https://github.com/craffael/lehrfempp/actions/runs/7586593522/job/20665126047#step:7:1381
Weisst du an was das liegt?
Leider weiss ich es nicht und konnte den Fehler heute Abend auch nicht finden. In den nächsten zwei Wochen habe ich noch Prüfungen. Ich werde mir das Problem danach genauer ansehen.
@craffael Das problem ist ein Fehler in den Test cases. In diesen wird zum testen jeweils die L2 norm im code approximiert und analytisch berechnet. Bei den Approximationen habe ich noch die Wurzel der norm genommen. Wahrscheinlich weil ich ursprünglich eine Funktion für die quadrierte Norm hatte. Die genauen zwei stellen habe ich unten vermerkt.
Weiter habe ich auch nachgeschaut ob ich die norm an anderen stellen im code auch noch gewurzelt habe. Aber dies ist nicht der Fall. Also der restliche code ist von dem Fehler nicht betroffen und sollte fehlerfrei laufen. Ich gehe davon aus dass mir der Fehler bei einem merge passiert, bei welchem ich womöglich die korrekten test mit den Fehlerhaften überschrieben habe.
Vielen Dank für deine Änderungen, leider bin ich mit dem ganzen "linting" nach wie vor noch nicht so vertraut, dies ist jedoch eine wunderbare Gelegenheit es zu lernen.
Die beiden Fehler befinden sich in: https://github.com/craffael/lehrfempp/blob/sphere_project/projects/hldo_sphere/post_processing/test/norms_tests.cc#L95 und https://github.com/craffael/lehrfempp/blob/sphere_project/projects/hldo_sphere/post_processing/test/norms_tests.cc#L171 wobei in beiden Zeilen die Wurzel zu entfernen ist, da der output der norm Funktion bereits die norm ist und nicht die quadrierte Norm.
Hoffe dass mir nicht noch weitere Fehler unterlaufen sind.
Nochmals vielen Dank!