0002762: Bauernwachstum fehlerhaft - MantisBT

ID	Projekt	Kategorie	Sichtbarkeit	Meldungsdatum	Zuletzt aktualisiert

0002762	Eressea		öffentlich	2021-06-19 11:27	2021-07-17 09:08

Reporter	Xolgrim	Bearbeitung durch	Enno
Priorität	normal	Schweregrad	kleinerer Fehler	Reproduzierbar	nicht getestet
Status	geschlossen	Lösung	erledigt
Produktversion	3.28
Zielversion	3.28	Behoben in Version	3.28

Zusammenfassung	0002762: Bauernwachstum fehlerhaft
Beschreibung	Deine Gedanken vom 12.03.2021 und 13.03.2021 aus DIscord habe ich unten angefügt. Du wolltest zur reduzierung von Zufallszahlen und der dadurch einhergehenden erleichterung für Tests und bugsuche (So habe ich das zumindest verstanden) das Bauernwachstum immer aufrunden. Regionen unter 1000 Bauern (Bei denen der Wachstumsboot für sehr Bauernarme regionen mit guten Burgen noch nicht greift) haben nun jedoch kein Wachstum mehr und bei allen anderen Regionen wird abgerundet und nicht aufgerundet. 1900 Bauern ergeben also z.B. +1 und nicht +2 Bauern
Zusätzliche Informationen	Enno — 12.03.2021 Darf ich mal sagen, dass ich makros wie diese hasse? #define RAND_ROUND(fractional) ((rng_double() < fractional-(int)fractional)?((int)fractional+1):((int)fractional)) Was hat sich der Autor dabei gedacht? Ein Herr Steffen Mecke war das in 2015. "implementing a much faster version of peasant growth calculation" Der hat da wohl auch die Sache mit dem injizieren gemacht, die ich eben noch gelobt habe. Nehme ich das Lob jetzt zurück? Immerhin wird das für nichts anderes als das Bauernwachstum benutzt. Der Test dafür benutzt allerdings nicht das mit dem injizieren, und würfelt einfach zufällig. Gelegentlich meldet er mal, dass was kaputt ist, aber nie reproduzierbar. Ich glaube, da gibt's nur eins: Schlafen gehen. huch. doch reproduzierbar. Wenn ich mit clang kompiliere. egal. Morgen. Enno — 13.03.2021 weg damit. Bauernwachstum wird immer aufgerundet, basta.

Partei	ii
Spiel	E2
Report	1223

Änderungsdatum	Benutzername	Feld	Änderung
2021-06-19 11:27	Xolgrim	Neuer Eintrag
2021-06-23 19:15	Enno	Bearbeitung durch	=> Enno
2021-06-23 19:15	Enno	Status	neu => zugewiesen
2021-06-23 19:24	Enno	Zielversion	=> 3.28
2021-06-23 19:25	Enno	Status	zugewiesen => erledigt
2021-06-23 19:25	Enno	Lösung	offen => erledigt
2021-06-23 19:25	Enno	Behoben in Version	=> 3.28
2021-06-23 19:25	Enno	Notiz hinzugefügt: 0009297
2021-07-17 09:08	Xolgrim	Status	erledigt => geschlossen