New Game does not read the seed input or set GameState.run_seed #2

New issue

Closed

opened 2026-08-15 11:36:19 +01:00 by liamjd · 0 comments

liamjd commented

2026-08-15 11:36:19 +01:00

Owner

scripts/main_menu.gd:

func _on_new_game_pressed() -> void:
	pass

The %SeedInput LineEdit collects a value that is never read, and GameState.run_seed is never written. The New Game button does nothing.

Two parts:

1. Resolve the seed text to an int. Agreed rule:

empty → random (randi()), matching the "random" placeholder
valid integer (String.is_valid_int()) → use it literally
anything else → hash(text), so word seeds like europa work

Avoid SpinBox for this: it stores its value as a float, so seeds above 2^53 lose precision, and it cannot express "blank means random".

2. Write it to GameState.run_seed, then change scene.

There is no game scene yet, so this issue can land as far as populating GameState and logging the resolved seed. The scene change follows once a game scene exists.

Depends on the run_seed default being fixed first, otherwise the two changes conflict.

`scripts/main_menu.gd`: ```gdscript func _on_new_game_pressed() -> void: pass ``` The `%SeedInput` `LineEdit` collects a value that is never read, and `GameState.run_seed` is never written. The New Game button does nothing. Two parts: **1. Resolve the seed text to an int.** Agreed rule: - empty → random (`randi()`), matching the "random" placeholder - valid integer (`String.is_valid_int()`) → use it literally - anything else → `hash(text)`, so word seeds like `europa` work Avoid `SpinBox` for this: it stores its value as a float, so seeds above 2^53 lose precision, and it cannot express "blank means random". **2. Write it to `GameState.run_seed`, then change scene.** There is no game scene yet, so this issue can land as far as populating `GameState` and logging the resolved seed. The scene change follows once a game scene exists. Depends on the `run_seed` default being fixed first, otherwise the two changes conflict.

liamjd referenced this issue from a commit

2026-08-15 11:58:40 +01:00

Randomise run_seed on startup instead of defaulting to 123

liamjd referenced this issue

2026-08-15 12:00:49 +01:00

Randomise run_seed on startup instead of defaulting to 123 #4

liamjd referenced this issue from a commit

2026-08-15 12:01:55 +01:00

Randomise run_seed on startup instead of defaulting to 123 (#4)

liamjd referenced this issue from a commit

2026-08-15 12:51:44 +01:00

Wire the seed input into GameState.run_seed