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

added optional seeding to gmtool.build_island again #1032

Open
wants to merge 29 commits into
base: develop
Choose a base branch
from

Conversation

stm2
Copy link
Member

@stm2 stm2 commented Sep 25, 2023

autoseed.c:build_island is able to seed factions from newfactions, but this feature was no longer accessible from gmtool or lua.

ennorehling and others added 28 commits August 27, 2023 20:38
fix resistance tests
fix null-pointer crash
some mild refactoring, but no real changes.
https://bugs.eressea.de/view.php?id=2980
Ausserdem:
Flammenschwerter vor Laenschwertern bevorzugen.
Co-authored-by: Enno Rehling <enno@eressea.de>
Co-authored-by: Enno Rehling <enno@eressea.de>
* narrow down failing test'

* on failure, print the filename
let install script use cmake --install
Co-authored-by: Enno Rehling <enno@eressea.de>
* upgrade to python3

* get rid of python2 compatibility

---------

Co-authored-by: Enno Rehling <enno@eressea.de>
@ennorehling
Copy link
Member

Ich setze meine Spieler gerne von Hand aus (mit 's'), wenn ich die Änderung richtig verstehe, werden die jetzt automatisch gesetzt, wenn eine Insel mit 'B' erzeugt wird? Manchmal will ich absichtlich eine leere Insel machen, oder ich mache eine Zufalls-Insel und terraforme da noch etwas dran herum. Die Flexibilität will ich nicht verlieren.

@stm2
Copy link
Member Author

stm2 commented Sep 25, 2023

Verstehe ich. Es gäbe verschiedene Workarounds. Man könnte newfactions löschen, wenn man nicht aussetzen will. Man könnte B einen zusätzlichen Parameter abfragen lassen. Man könnte auch aus autoseed einen separaten Befehl machen. Letzteres ist wohl am meisten Arbeit. Was wäre dir am liebsten?

@ennorehling
Copy link
Member

Von mir aus gerne zwei verschiedene Hotkeys (kleines b für leere Inseln, großes B für Inseln mit Parteien). Historisch ist das B übrignes für einen Block: die Welt war mal in quadratische Blöcke unterteilt, ehe sie seckseckig wurde, im Editor findet man davon noch ein paar Überbleibsel (und in der ersten Welt ein Gittermuster).

@ennorehling ennorehling self-requested a review October 9, 2023 07:28
Copy link
Member

@ennorehling ennorehling left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bitte für auto-seeding bei Erzeugen einer Insel einen eigenen Hotkey einführen, und die alte Funktionalität beibehalten.

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