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

Switch zwischen FHN und GrayScott einbauen #5

Merged
2211567 merged 14 commits from feat/common_ui into main 2025-06-17 09:56:58 +02:00
Conversation 0 Commits 14 Files Changed 6
2211567 commented 2025-06-15 21:41:41 +02:00
Collaborator

Ich hab jetzt nen groben Switch mit nem Dropdown-Menü eingebaut.

Wichtige Anmerkungen

  • Makie ist echt schlimm mit Responsivität. Deswegen werden jetzt einfach alle Eingabefelder immer angezeigt
Ich hab jetzt nen groben Switch mit nem Dropdown-Menü eingebaut. **Wichtige Anmerkungen** - Makie ist echt schlimm mit Responsivität. Deswegen werden jetzt einfach alle Eingabefelder immer angezeigt
2211567 added 8 commits 2025-06-15 21:41:41 +02:00
2211567 added 1 commit 2025-06-16 15:36:06 +02:00
2211567 added 1 commit 2025-06-16 15:42:00 +02:00
2212719 reviewed 2025-06-16 20:06:59 +02:00
src/visualization.jl Outdated
@ -143,2 +172,4 @@
heat_obs[] = copy(U)
end
on(dropdown.selection) do sel
2212719 commented 2025-06-16 20:06:59 +02:00
Owner

Wechselt immer vom gleichen Model zum Gleichen.

Wechselt immer vom gleichen Model zum Gleichen.
2212719 added 2 commits 2025-06-16 20:44:50 +02:00
2212719 added 1 commit 2025-06-16 20:49:42 +02:00
2212719 requested changes 2025-06-16 21:32:55 +02:00
scripts/main.jl
@ -7,3 +4,3 @@
using GLMakie
using .Constants
using .Visualization.Constants
2212719 commented 2025-06-16 21:26:33 +02:00
Owner

wieso hast du hier Visualization vor das Constants geschrieben? Macht es nicht mehr Sinn Constants selbst zu importieren? Oder bei Namenskonflikten explizit das Constants Modul anzugeben. Mit dem Doppelnamen sieht es nicht so schön aus finde ich

wieso hast du hier Visualization vor das Constants geschrieben? Macht es nicht mehr Sinn Constants selbst zu importieren? Oder bei Namenskonflikten explizit das Constants Modul anzugeben. Mit dem Doppelnamen sieht es nicht so schön aus finde ich
2211567 commented 2025-06-16 21:53:13 +02:00
Poster
Collaborator

Weil Julia leider unterscheidet zwischen Main.Constants und Visualization.Constant

Weil Julia leider unterscheidet zwischen Main.Constants und Visualization.Constant
src/fhn_solver.jl
@ -1,3 +1,5 @@
module FHNSolver
2212719 commented 2025-06-16 21:29:29 +02:00
Owner

Für die beiden Module FHN und GrayScott, fänd ichs eig nicer, wenn wir multi dispatching implementieren könnten. Also dass wir einfach die step! Methode. Und je nachdem was für ein params_obs man reingibt, verhält sie sich anders. Also wenn es ein GSParams ist oder ein FHNParams.
step!(p::GSParams...)
step!(p::FHNParams...)
Wenn das aber zu schwierig ist, weil es diesen switch gibt, dann lassen wir es so. Wäre halt der more juliane way I guess

Für die beiden Module FHN und GrayScott, fänd ichs eig nicer, wenn wir multi dispatching implementieren könnten. Also dass wir einfach die step! Methode. Und je nachdem was für ein params_obs man reingibt, verhält sie sich anders. Also wenn es ein GSParams ist oder ein FHNParams. step!(p::GSParams...) step!(p::FHNParams...) Wenn das aber zu schwierig ist, weil es diesen switch gibt, dann lassen wir es so. Wäre halt der more juliane way I guess
2211567 commented 2025-06-16 21:52:28 +02:00
Poster
Collaborator

Ich setz mich mal dran

Ich setz mich mal dran
src/visualization.jl
@ -69,23 +93,28 @@ function build_ui(U, V, param_obs_map::NamedTuple, params_obs, heat_obs)
2212719 commented 2025-06-16 20:27:09 +02:00
Owner 
slidergrid = SliderGrid(fig[3, 1], (label="Speed", range=1:1:100, format="{}x", width=350, startvalue=stepsize[]), tellwidth=false)

in Zeile 92 tellwidth=false dazu

slidergrid = SliderGrid(fig[3, 1], (label="Speed", range=1:1:100, format="{}x", width=350, startvalue=stepsize[]), tellwidth=false) in Zeile 92 tellwidth=false dazu
2211567 marked this conversation as resolved
src/visualization.jl Outdated
@ -102,7 +134,7 @@ function build_ui(U, V, param_obs_map::NamedTuple, params_obs, heat_obs)
end
2212719 commented 2025-06-16 21:32:42 +02:00
Owner

kannst du hier an das stepsize[] noch einen Multiplikator dranmachen? Einfach so *5. Sonst sieht man immer nicht, dass es stept

kannst du hier an das stepsize[] noch einen Multiplikator dranmachen? Einfach so *5. Sonst sieht man immer nicht, dass es stept
2211567 marked this conversation as resolved
src/visualization.jl Outdated
@ -115,3 +143,4 @@
multi_step!((U, V), stepsize[], heat_obs, params_obs; step_method=step_method[])
sleep(0.0015) # ~20 FPS
end
end
2212719 commented 2025-06-16 20:09:00 +02:00
Owner

Da unten den print gerne raus. Der ist von mir gewesen mit params obs

Da unten den print gerne raus. Der ist von mir gewesen mit params obs
2211567 added 1 commit 2025-06-16 21:54:09 +02:00
2211567 merged commit efed0a47e0 into main 2025-06-17 09:56:58 +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#5
There is no content yet.
Delete Branch "feat/common_ui"

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?