Eintragsdetails ansehen

IDProjektKategorieSichtbarkeitZuletzt aktualisiert
0002396EresseaREKRUTIEREöffentlich2018-07-08 12:35
ReporterJuliusBearbeitung durchEnno 
PrioritätnormalAuswirkungkleinerer FehlerReproduzierbarnicht getestet
Status erledigtLösungerledigt 
Produktversion3.14.2 
Zielversion3.15Behoben in Version3.15 
Zusammenfassung0002396: Insekten können in letzer Herbstwoche scheinbar nicht rekrutieren
Beschreibung

Laut nr-Report ist diese Woche die letzte, in der Insekten rekrutieren können. Aktuell ist die zweite Woche des Monats Sturmmond. Nächste Woche ist also die dritte Woche des Monats Sturmmond und daher auch noch Herbst! Laut Regeln können Insekten im Winter nicht rekrutieren

Für meine noch junge 3-Wochen alte Insekten Partei ist das ein Problem, da ich 5 Wochen eingeplant hatte, in denen ich rekrutieren darf.

Zusätzliche Informationen

Report für Eressea, Saturday, 16. December 2017, 21:21
Wir schreiben die zweite Woche des Monats Sturmmond im Jahre 33 des zweiten Zeitalters. Es ist Herbst.
[...]
Es ist Spätherbst, und diese Woche ist die letzte vor dem Winter, in der Insekten rekrutieren können.

Parteisfs
SpielE2
Report1055

Eintrags-Beziehungen

verwandt mit 0002389 erledigtEnno Schiffe mit Insekten an Bord können nicht in einen Gletscher segeln. 
verwandt mit 0002458 zugewiesenEnno Insekten konnten in letzter Herbstwoche nicht rekrutieren! 

Notizen / Dateien

Enno

Enno

2017-12-21 09:46

Administrator   ~0007696

Ich sagte doch, dass die Jahreszeiten verwirrend sind. @Xolgrim meinte, die seien total logisch, aber für mich als Programmierer sind sie das nicht, weil die Auswertung immer schon die folgende Runde betrachtet, nicht die im NR angegebene. Seufz. Vielleicht ist das die Lösung? Die turn Variable erst für den Report anheben, nicht vor der AW schon? Das macht eine Menge Tests kaputt, und muss vorsichtig eingeführt werden, aber ich glaube, es wäre eine gute Lösung.

Enno

Enno

2017-12-21 09:47

Administrator   ~0007697

Zur Information: Es sollte momentan gelten, dass man als Insekt rekrutieren kann, wenn im Report nicht steht, dass es Winter ist. Die Meldung ist also wahrscheinlich einfach falsch.

Enno

Enno

2017-12-21 09:48

Administrator   ~0007698

http://www.catb.org/jargon/html/O/off-by-one-error.html

Enno

Enno

2017-12-21 10:34

Administrator   ~0007699

Fehler gefunden, den kann ich sicher ohne große Umbauten reparieren. Hier wird 1 Woche zu viel addiert:

get_gamedate(turn + 1, &date);
thisseason = date.season;
get_gamedate(turn + 2, &date);
nextseason = date.season;

Das gilt nur für die Anzeige der Warnung, glaube ich.

Enno

Enno

2017-12-21 10:47

Administrator   ~0007700

Was bei dieser Gelegenheit auch mal gemacht werden könnte: Dies ist eine der Warnungen, die nur im NR stehen, und die Magellan-Benutzer deshalb nie zu sehen bekommen.

Enno

Enno

2017-12-22 11:16

Administrator   ~0007701

Zusammen mit der Immunität vor Attacken sollte das demnächst in NR und CR auftauchen (und richtig ist es auch).

Eintrags-Historie

Änderungsdatum Benutzername Feld Änderung
2017-12-16 21:38 Julius Neuer Eintrag
2017-12-19 11:50 Solthar Beziehung hinzugefügt verwandt mit 0002389
2017-12-21 09:46 Enno Notiz hinzugefügt: 0007696
2017-12-21 09:46 Enno Bearbeitung durch => Enno
2017-12-21 09:46 Enno Status neu => zugewiesen
2017-12-21 09:47 Enno Notiz hinzugefügt: 0007697
2017-12-21 09:48 Enno Notiz hinzugefügt: 0007698
2017-12-21 10:34 Enno Notiz hinzugefügt: 0007699
2017-12-21 10:47 Enno Notiz hinzugefügt: 0007700
2017-12-22 11:16 Enno Status zugewiesen => erledigt
2017-12-22 11:16 Enno Lösung offen => erledigt
2017-12-22 11:16 Enno Behoben in Version => 3.15
2017-12-22 11:16 Enno Notiz hinzugefügt: 0007701
2017-12-22 11:16 Enno Produktversion => 3.14.2
2017-12-22 11:16 Enno Zielversion => 3.15
2018-07-08 12:35 Enno Beziehung hinzugefügt verwandt mit 0002458