Jonathan/Babushka
2
0
Fork 0
Code Pull Requests 1 Releases Activity

Randomized who starts the fight #41

Merged
Jonathan merged 1 commits from randomise_fight_start into develop 2025-12-16 19:28:38 +01:00
Conversation 3 Commits 1 Files Changed 2
+22
Jonathan commented 2025-12-09 15:09:36 +01:00
Owner
Copy Link
Copy Source

See title

See title
Jonathan added 1 commit 2025-12-09 15:09:36 +01:00
Randomized who starts the fight a736adaafb
Jonathan requested review from kziolkowski 2025-12-09 15:09:45 +01:00
kziolkowski reviewed 2025-12-11 22:54:15 +01:00
scripts/CSharp/Common/Fight/FightHappening.cs
@@ -308,2 +308,4 @@
HappeningData.fighterTurn.AddAsLast(fighter);
}
if (GD.RandRange(0, 2) != 0) // 2/3 chance for vesna to start
kziolkowski commented 2025-12-11 22:54:15 +01:00
Owner
Copy Link
Copy Source

warum nicht 50/50?

warum nicht 50/50?
Jonathan commented 2025-12-16 19:26:10 +01:00
Author
Owner
Copy Link
Copy Source

Wirkt "fairer" für den Spieler

Wirkt "fairer" für den Spieler
kziolkowski reviewed 2025-12-11 22:55:39 +01:00
scripts/CSharp/Common/Fight/FighterTurn.cs
@@ -101,1 +101,4 @@
/// <summary>
/// Sets the current one back
/// This is an expensive operation, because the entire data structure needs to be circled
kziolkowski commented 2025-12-11 22:55:39 +01:00
Owner
Copy Link
Copy Source

Danke für den Hinweis. Passiert das aktuell nur beim Kampfstart oder könnte das potenziell auch wann anders benutzt werden? Und wenn ja, ab wann würde das zu Problemen führen?

Danke für den Hinweis. Passiert das aktuell nur beim Kampfstart oder könnte das potenziell auch wann anders benutzt werden? Und wenn ja, ab wann würde das zu Problemen führen?
Jonathan commented 2025-12-16 19:28:31 +01:00
Author
Owner
Copy Link
Copy Source

Passiert aktuell nur beim Kampfstart.
Probleme sollte das nie machen. Dafür müssten Tausende oder sogar Millionen Kämpfer aktiv sein, bevor man das performancetechnisch merkt.

Passiert aktuell nur beim Kampfstart. Probleme sollte das nie machen. Dafür müssten Tausende oder sogar Millionen Kämpfer aktiv sein, bevor man das performancetechnisch merkt.
kziolkowski approved these changes 2025-12-11 22:55:42 +01:00
Jonathan merged commit edc133749f into develop 2025-12-16 19:28:38 +01:00
Sign in to join this conversation.
Reviewers
No Reviewers
kziolkowski
Labels
No items
No Label
Milestone
No items
No Milestone
Assignees
Clear assignees
ascheck (Anna Scheck) jlink kziolkowski (Katharina Ziolkowski) lkrause Nao Palurdas saderako Sanelschnitte Jonathan
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Jonathan/Babushka#41
Delete Branch "randomise_fight_start"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?