Eintragsdetails ansehen

IDProjektKategorieSichtbarkeitZuletzt aktualisiert
0002762Eresseaöffentlich2021-07-17 09:08
ReporterXolgrim Bearbeitung durchEnno  
PrioritätnormalSchweregradkleinerer FehlerReproduzierbarnicht getestet
Status geschlossenLösungerledigt 
Produktversion3.28 
Zielversion3.28Behoben in Version3.28 
Zusammenfassung0002762: 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.

Parteiii
SpielE2
Report1223

Notizen / Dateien

Enno

Enno

2021-06-23 19:25

Administrator   ~0009297

simpler fix

Eintrags-Historie

Ä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