Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sphere project #209

Open
wants to merge 233 commits into
base: master
Choose a base branch
from
Open

Sphere project #209

wants to merge 233 commits into from

Conversation

craffael
Copy link
Owner

No description provided.

@craffael
Copy link
Owner Author

@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
Copy link

ncograf commented Jan 19, 2024

@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.

@ncograf
Copy link

ncograf commented Feb 1, 2024

@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!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants