View Issue Details

IDProjectCategoryView StatusLast Update
0002164EresseaGeneralpublic2017-12-05 18:49
ReporterSuperEcho Assigned ToEnno  
PrioritynormalSeverityminorReproducibilityN/A
Status closedResolutionfixed 
Target Version3.7.3Fixed in Version3.7.3 
Summary0002164: Schiffe nehmen beim Abtreiben durch Stürme keinen Schaden
Description

Unter "Ereignisse" steht im Report: Die OdS Becquerel (2tg8) wird in Ozean (-38,28) von Stürmen abgetrieben.

Das Schiff ist allerdings völlig unbeschädigt: OdS Becquerel (2tg8), Karavelle, (2820/3000).

TagsNo tags attached.
Factions19L
GameE2
Report955

Relationships

related to 0002214 closedEnno Abtreibende Schiffe nehmen Schaden 
related to 0002223 closedEnno Schiff treibt ab obwohl und nimmt Schaden obwohl beides nicht passieren dürfte 

Activities

Enno

Enno

2015-11-22 16:28

administrator   ~0006298

Im Code wird der Schaden direkt nach Erzeugung der im Bugreport angegebenen Nachricht zugefügt:

ADDMSG(&f->msgs, msg_message("storm", "ship region sink",
    sh, current_point, sh->damage >= sh->size * DAMAGE_SCALE));

/* damage the ship. we handle destruction in the end */
damage_ship(sh, damage_drift());

Allerdings steht in config.json rules.ship.damage_drift = 0.0, statt des defaults von 0.02 das erklärt den Bug dann wohl?

Enno

Enno

2015-11-22 17:06

administrator   ~0006299

Zwar nicht reproduzierbar, weil Zufallsereignis, aber bei einer Neuauswertung mit Bugfix treibt ein anderes Schiff ab, und scheint Schaden zu haben:

Die OdS Hewish (zaqh) wird in Ozean (21,-7) von Stürmen abgetrieben. OdS Hewish (zaqh), Karavelle, (40/2970), 1% beschädigt.

Allerdings ist 1% ja nicht 2%, ist das ein Rundungsfehler? Sollte ich noch mal nachrechnen, das alles.

Solthar

Solthar

2015-11-22 23:49

developer   ~0006301

Ja, das war offenbar schon immer so. Den Rundungsfehler habe ich in meinem massive_overload-Patch nebenbei behoben (und du hast das kommentiert).

Enno

Enno

2015-11-23 15:54

administrator   ~0006319

https://github.com/eressea/server/pull/407

Enno

Enno

2015-11-23 17:29

administrator   ~0006323

CTD hat mir gerade geschrieben, dass das wohl Absicht war, und eine Angleichung von E2 auf E3 - weniger Kleinschaden durch Abtreibung führt zu weniger Mikromanagement bei der Reparatur von Schiffen, sagt er. Wenn wir das wirklich geplant hatten, dann habe ich es vergessen, aber Sinn macht es wohl.

Enno

Enno

2015-11-23 18:12

administrator   ~0006324

https://github.com/eressea/server/pull/408

Ich habe den Code für den Abtreibe-Schaden einfach komplett entfernt. Spurlos verschwunden, dann verwirrt er mich auch in der Zukunft nicht mehr.

Issue History

Date Modified Username Field Change
2015-11-21 22:00 SuperEcho New Issue
2015-11-22 16:28 Enno Note Added: 0006298
2015-11-22 17:06 Enno Note Added: 0006299
2015-11-22 17:06 Enno Assigned To => Enno
2015-11-22 17:06 Enno Status new => assigned
2015-11-22 18:44 Enno Target Version => 3.7.3
2015-11-22 23:49 Solthar Note Added: 0006301
2015-11-23 15:54 Enno Note Added: 0006319
2015-11-23 15:54 Enno Status assigned => resolved
2015-11-23 15:54 Enno Fixed in Version => 3.7.3
2015-11-23 15:54 Enno Resolution open => fixed
2015-11-23 17:29 Enno Note Added: 0006323
2015-11-23 17:29 Enno Status resolved => feedback
2015-11-23 17:29 Enno Resolution fixed => reopened
2015-11-23 17:30 Enno Status feedback => assigned
2015-11-23 18:12 Enno Note Added: 0006324
2015-11-23 18:12 Enno Status assigned => resolved
2015-11-23 18:12 Enno Resolution reopened => fixed
2016-06-17 21:45 Solthar Relationship added related to 0002214
2016-07-31 08:17 Xolgrim Relationship added related to 0002223
2017-12-05 18:49 Enno Status resolved => closed