View Issue Details

Jump to NotesJump to History
IDProjectCategoryView StatusDate SubmittedLast Update
0001808EresseaGeneralpublic2010-09-28 10:402011-05-10 04:31
ReporterSolthar Assigned ToEnno  
PrioritynormalSeveritymajorReproducibilityalways
Status closedResolutionfixed 
Summary0001808: LEHRE offenbar kaputt
Description

LEHRE scheint nicht zu funktionieren. Dies wurde mir gegenüber auch von anderen Parteien vermutet.

Additional Information

Einheit Segler (o2r2) brauchte von Runde 675 bis 685 um Unterhaltung von 7 auf 8 zu kommen. Dies dürfte mit Lehrer aber höchstens 8 Runden dauern.

TagsNo tags attached.
Faction1wpy
Game
Report675

Relationships

has duplicate 0001804 closedEnno Fehler bei Lehre 

Activities

Solthar

Solthar

2010-09-28 10:43

developer   ~0004614

Ich verstehe den Code nicht 100%, aber wenn es kein Konfigurationsfehler ist, könnte es hieran liegen:

diff --git a/shared/src/gamecode/study.c b/shared/src/gamecode/study.c index 32911c2..0d3debb 100644 --- a/shared/src/gamecode/study.c +++ b/shared/src/gamecode/study.c @@ -197,7 +197,7 @@ teach_unit(unit teacher, unit student, int nteaching, skill_t sk, return 0; }

  • n = 30;
  • n = 30 student->number; a = a_find(student->attribs, &at_learning); if (a!=NULL) { teach = (teaching_info)a->data.v;
Enno

Enno

2010-10-06 03:31

administrator   ~0004617

Ich habe das vor einer Weile shconmal angesehen, und glaube auch, dass das so sein sollte. Der Code ist ziemlich kompliziert, ja.
Enno

Enno

2010-10-06 05:14

administrator   ~0004618

Der LERNE/LEHRE code ist nicht nur eklig, er ist auch grauenhaft unter Tests zu bekommen, wegen dem ganzen verfluchten Zufallskrempel. Ich habe mal den Multiplikator eingebaut, und einen rudimentären Test drumherum gestrickt, bin aber nicht glücklich mit dem Resultat. Refactoring dringend notwendig.
Fiete

Fiete

2010-10-08 16:16

reporter   ~0004619

Besteht doch Relevanz zum Bug 1804...oder? http://bugs.eressea.de/view.php?id=1804
Enno

Enno

2010-10-09 06:41

administrator   ~0004620

Ich habe raus, wie ich das vernünftig testen kann, und das ist implementiert und verifiziert. LEHRE sollte also wieder gehen.

Issue History

Date Modified Username Field Change
2010-09-28 10:40 Solthar New Issue
2010-09-28 10:43 Solthar Note Added: 0004614
2010-10-06 03:31 Enno Assigned To => Enno
2010-10-06 03:31 Enno Status new => assigned
2010-10-06 03:31 Enno Note Added: 0004617
2010-10-06 05:14 Enno Note Added: 0004618
2010-10-08 16:16 Fiete Note Added: 0004619
2010-10-09 06:41 Enno Note Added: 0004620
2010-10-09 06:41 Enno Status assigned => resolved
2010-10-09 06:41 Enno Resolution open => fixed
2010-10-09 06:42 Enno Relationship added has duplicate 0001804
2011-05-10 04:31 Enno Status resolved => closed