View Issue Details

Jump to NotesJump to History
IDProjectCategoryView StatusDate SubmittedLast Update
0002239EresseaGeneralpublic2016-09-19 15:482017-03-02 07:13
ReporterXolgrim Assigned ToEnno  
PrioritylowSeverityminorReproducibilityhave not tried
Status closedResolutionfixed 
Fixed in Version3.10.0 
Summary0002239: Neu erschaffener Schneemann ist schwer verwundet
Description

Ein in Runde 988 erzeugter und damit in 989 erstmals im Report stehender Scheemann war nach seiner Erschaffung schwer verwundet. Entweder ist das so gewollt gewesen, damit man die nicht bei gegnern in Gletscher stellt und die sofort "kampfbereit" sind oder aber das ganze ist ein Fehler. Falls letzteres der Fall ist, nur so ein Denkanstoß: Erschaffen wurde der Schneemann von einem Zwergen (24HP) Schneemänner haben laut zeige 1000HP. Eventuell geht er bei der Erschaffung von den 24HP aus und wird dann erst später auf 1000 hoch geschraubt. 24 von 1000 wäre ganz klar "Schwer verwundet"

Additional Information

Region: "Ronkot" Reg.ID: 2088688474

Runde 988:

EINHEIT ybst; Wächter der Elemente [1,0$] kämpft nicht LERNE Wahrnehmung benutze 1 schneemann

Runde 989

  • Einheit Li2y (Li2y), 1 Erzschneemann, aggressiv (schwer verwundet), "ARBEITE".
TagsNo tags attached.
Faction777
GameE2
Report989

Activities

Enno

Enno

2016-10-02 08:18

administrator   ~0006787

Gerade mal schnell den Test in e2features.lua erweitert, und der sagt, ein neuer Schneemann hat 70 HP. Hat also mit Zwergen-HP ganz bestimmt nichts zu tun.
Enno

Enno

2016-10-02 09:28

administrator   ~0006788

Meine vorherige Aussage ist Unsinn, denn der Test benutzt Menschen, keine Zwerge.

Aber: use_snowman erzeugt zuerst eine Einheit mit der Parteirasse, und ändert dann nachträglich deren Rasse. Das ändert aber nicht die Trefferpunkte, und ist generell eine idiotisch Art, das zu machen.
Enno

Enno

2016-10-02 09:53

administrator   ~0006789

gefixt in https://github.com/eressea/server/pull/589
Enno

Enno

2016-10-02 11:47

administrator   ~0006790

Auf dem Mac kriege ich jetzt einen Fehler:

1) Failure (tests.e2.e2features.test_snowman): ./scripts/tests/e2/e2features.lua:219: expected 1000 but was 70

Das ist sehr seltsam.
Enno

Enno

2016-10-02 11:49

administrator   ~0006791

Schlimmer, da ist noch ein anderer Test unzuverlässig:

1) Failure (tests.xmas.test_snowglobe): ./scripts/tests/xmas.lua:61: expected 4 but was 5

2) Failure (tests.e2.e2features.test_snowman): ./scripts/tests/e2/e2features.lua:219: expected 1000 but was 70
Enno

Enno

2016-10-02 12:34

administrator   ~0006792

Klassischer Fall von PEBCAK. Ich hatte nicht die aktuelle Version des Code, wegen lokaler Aenderungen hat git sich geweigert. Lange shell-history zu haben lohnt sich:

c53ac9d..fadf591 develop -> origin/develop Updating c53ac9d..fadf591 error: Your local changes to the following files would be overwritten by merge: scripts/tests/e2/e2features.lua Please commit your changes or stash them before you can merge.

Das war's dann.

Issue History

Date Modified Username Field Change
2016-09-19 15:48 Xolgrim New Issue
2016-10-02 08:18 Enno Note Added: 0006787
2016-10-02 09:26 Enno Assigned To => Enno
2016-10-02 09:26 Enno Status new => assigned
2016-10-02 09:28 Enno Note Added: 0006788
2016-10-02 09:53 Enno Status assigned => resolved
2016-10-02 09:53 Enno Resolution open => fixed
2016-10-02 09:53 Enno Fixed in Version => 3.10.0
2016-10-02 09:53 Enno Note Added: 0006789
2016-10-02 11:47 Enno Status resolved => feedback
2016-10-02 11:47 Enno Resolution fixed => reopened
2016-10-02 11:47 Enno Note Added: 0006790
2016-10-02 11:49 Enno Note Added: 0006791
2016-10-02 12:34 Enno Status feedback => resolved
2016-10-02 12:34 Enno Resolution reopened => fixed
2016-10-02 12:34 Enno Note Added: 0006792
2017-03-02 07:13 Enno Status resolved => closed