2212719
/
SCJ_Projekt
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

FHN mit neuer UI kompatibel machen #4

Merged
2211567 merged 12 commits from feat/fhn_expansion into main 2025-06-14 22:11:49 +02:00
Conversation 0 Commits 12 Files Changed 9
2211567 commented 2025-06-14 14:53:31 +02:00
Collaborator

FHN ist jetzt kompatibel mit der UI, die Nikola eingeführt hatte. Man muss es gerade aber noch im Code ein- und auskommentieren. Ich mach das jetzt im "Folgeticket", dass das über die UI auch geht.

Sonstige Anmerkungen:

  • Man muss entsprechende Kommentare in der main.jl und visualizations.jl anpassen, damit man zu GrayScott wechseln kann
  • template.jl wird im Folgeticket erst benutzt
  • Es muss noch geschaut werden, ob man zB einen universellen solver implementieren kann und die laplacians in eins zusammengefügt werden kann. Noch ziemlich Baustelle alles
FHN ist jetzt kompatibel mit der UI, die Nikola eingeführt hatte. Man muss es gerade aber noch im Code ein- und auskommentieren. Ich mach das jetzt im "Folgeticket", dass das über die UI auch geht. **Sonstige Anmerkungen:** - Man muss entsprechende Kommentare in der main.jl und visualizations.jl anpassen, damit man zu GrayScott wechseln kann - template.jl wird im Folgeticket erst benutzt - Es muss noch geschaut werden, ob man zB einen universellen solver implementieren kann und die laplacians in eins zusammengefügt werden kann. Noch ziemlich Baustelle alles
2211567 added 11 commits 2025-06-14 14:53:31 +02:00
2212719 approved these changes 2025-06-14 19:15:27 +02:00
2212719 left a comment
Owner

Nice, dass du das mit dem Params observable als Übergabe für die Ui hinbekommen hast. Dann muss man die Parameter nicht einzeln übergeben

Nice, dass du das mit dem Params observable als Übergabe für die Ui hinbekommen hast. Dann muss man die Parameter nicht einzeln übergeben
scripts/main.jl Outdated
@ -25,3 +55,3 @@
heat_obs = Observable(U)
fig = build_ui(U, V, Du, Dv, F, k, params_obs, heat_obs)
fig = Visualization.build_ui(U, V, param_observables, params_obs, heat_obs)
2212719 commented 2025-06-14 19:02:52 +02:00
Owner

hier wärs ganz nice, wenn du Visualization nicht extra nennen müsstest, weil du es ja oben eig schon includest. falls es was wegen ambiguity sagt, vllt die repl neustarten. Sollte aber klappen

hier wärs ganz nice, wenn du Visualization nicht extra nennen müsstest, weil du es ja oben eig schon includest. falls es was wegen ambiguity sagt, vllt die repl neustarten. Sollte aber klappen
2211567 marked this conversation as resolved
src/utils/constants.jl
@ -1,6 +1,8 @@
module Constants
struct FHNParams
abstract type PDEParams end
2212719 commented 2025-06-14 19:05:29 +02:00
Owner

was ist die Idee, dahinter, dass beide von PDE abstammen? Weil verwendet wird es momentan nicht. Damit man später bei Funktionensignaturen diesen Typen angeben kann?

was ist die Idee, dahinter, dass beide von PDE abstammen? Weil verwendet wird es momentan nicht. Damit man später bei Funktionensignaturen diesen Typen angeben kann?
2211567 commented 2025-06-14 22:11:32 +02:00
Poster
Collaborator

Genau, die wurde auch benutzt aber während der Implementierung dann doch durch n Observable ersetzt. Ich lass es trotzdem mal. Kann nicht schaden

Genau, die wurde auch benutzt aber während der Implementierung dann doch durch n Observable ersetzt. Ich lass es trotzdem mal. Kann nicht schaden
src/utils/laplacian.jl Outdated
@ -15,10 +15,11 @@ module Laplacian
- `Vector{Float64}`: A flattened (vectorized) representation of the approximated Laplacian values for each element in `U`. The boundary conditions are handled circularly.
"""
function laplacian(U::Matrix{Float64}, N::Int, h::Float64)
2212719 commented 2025-06-14 19:07:24 +02:00
Owner

Können wir die gleiche Laplacian Method für beide Modelle verwenden? Solange es momentan funktioniert, müssen wir uns keine Gedanken drüber machen. Aber wäre vllt nicht schlect. Aber low prio

Können wir die gleiche Laplacian Method für beide Modelle verwenden? Solange es momentan funktioniert, müssen wir uns keine Gedanken drüber machen. Aber wäre vllt nicht schlect. Aber low prio
2211567 commented 2025-06-14 22:10:51 +02:00
Poster
Collaborator

Done

Done
2211567 marked this conversation as resolved
2211567 added 1 commit 2025-06-14 22:10:21 +02:00
2211567 merged commit 8c07a4c969 into main 2025-06-14 22:11:49 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
2212719
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: 2212719/SCJ_Projekt#4
There is no content yet.
Delete Branch "feat/fhn_expansion"

Deleting a branch is permanent. Although the deleted branch may exist for a short time before cleaning up, in most cases it CANNOT be undone. Continue?