Fix TP/KK melee modifier caculations; update WdS handout URL; etc.

[davidfoerster]

TP/KK modifiers for melee weapon damage, AT, and PA are currently incorrectly rounded towards negative infinity instead of zero as required by the rule book. This PR fixes the issue and couple of others (see commit messages).

[flyx]

Kannst du die relevante Regelstelle zu TP/KK angeben? Ich finde im WdS 81-82:

Jeder Waffe sind daher zwei Werte zugeordnet: der Schwellenwert, der angibt, bei welcher Körperkraft ein Held ‘balanciert’ kämpft, und die Schadensschritte, die besagen, bei wie vielen weiteren Punkten KK es einen zusätzlichen Trefferpunkt (oder gar weitere zusätzliche) gibt. […]

Umgekehrt heißt dies aber auch, dass für fehlende Körperkraft-Punkte unter dem Schwellenwert auch Trefferpunkte von der Schadenswirkung abgezogen werden können. Bei obiger Waffe mit TP/KK 12/2 bedeutet dies, dass ein Kämpfer mit KK 10 einen TP weniger anrichtet, bei einer KK von 8 sogar zwei Punkte weniger.

Da steht nicht, wo hin gerundet wird, aber schon ein Punkt gilt doch als „fehlender Körperkraft-Punkt unter dem Schwellenwert“, oder? Warum würde er sonst „Schwellenwert“ heißen? Die gegebenen Beispiele klären die Frage nicht aber ich verstehe, dass man sie so lesen kann, dass erst ab diesem Wert ein Malus einsetzt.

Ich werd auch mal schauen was die Heldensoftware macht; vermutlich ist es sinnvoll, dasselbe zu machen.

[flyx]

In der Heldensoftware nachgeschaut, die macht das auch so wie der PR.

Sieht alles gut aus, danke!

[flyx]

Okay ich hätte das erst testen sollen. Der PR hat mehrere Probleme:

• Deine Änderung an der WdS-Handout-URL ist falsch (das & wird schon von Nix mit einem %26 ersetzt)
• Deine Änderung an der NiceMatrix ist falsch: color-inside gibt es nicht in der Version von TeX Live die NixOS gerade beinhaltet (2022). Auch wenn ich auf 23.11 update gibt es sie noch nicht.

Hast du das getestet?

[davidfoerster]

• Deine Änderung an der WdS-Handout-URL ist falsch (das & wird schon von Nix mit einem %26 ersetzt)

Oh, tut mir Leid. Ich bin nicht vertraut mit Nix und hatte den Link provisorisch per Hand eingefügt.

(Ehrlich gesagt finde ich das Vorgehen von Nix hier zweifelhaft. & und %26 ergeben verschiedene URLs, auch wenn viele Webserver sie in diesem Kontext als gleichwertig behandeln.)

• Deine Änderung an der NiceMatrix ist falsch: color-inside gibt es nicht in der Version von TeX Live die NixOS gerade beinhaltet (2022). Auch wenn ich auf 23.11 update gibt es sie noch nicht.

Mist. Ich dachte, ich hätte TeX Live 2023 im Nix-Repo gesehen und es deshalb hier installiert. Soll ich die Änderung „für später“ in ein separates PR ausgliedern, wenn Nix auf Tex Live ≥2023 wechselt?

Hast du das getestet?

Leider nicht mit Nix, weil ich das Flake hier nicht zum Laufen bekommen hatte. Deswegen hatte ich abgesehen von der URL nichts zum PR hinzugefügt, das (meines Wissens) von Nix abhängt.

[flyx]

Ehrlich gesagt finde ich das Vorgehen von Nix hier zweifelhaft

Ja, ich hab auch echt gebraucht bis ich raus hatte wie der Link „richtig“ im Flake zu stehen hat. Versteh nicht warum es das tut.

Mist. Ich dachte, ich hätte TeX Live 2023 im Nix-Repo gesehen

Ist es auch, halt in unstable. Das hab ich ungern als Abhängigkeit, wenn ich nicht muss. Im Mai wirds nach stable kommen.

Hast du das getestet?

Leider nicht mit Nix, weil ich das Flake hier nicht zum Laufen bekommen hatte.

Mh, vielleicht ein guter Zeitpunkt um mal ein CI aufzusetzen, das das automatisch testet. Edit: Done.

---

Kannst du nochmal einen PR nur mit den inhaltlichen Änderungen (letzte beiden commits) machen? Das color-inside kannst du dann gern nochmal schicken, wenn 24.05 draußen ist.

Allgemein wärs arg geschickt wenn du das Flake zum Laufen kriegst, weil das ist halt das Dependency Management Werkzeug das solche Probleme vermeiden soll.

[davidfoerster]

Kannst du nochmal einen PR nur mit den inhaltlichen Änderungen (letzte beiden commits) machen?

Ich nehme an, du meinst die ersten beiden Commits. Mache ich.

[davidfoerster]

Natürlich kann ich keinen PR mehr für sie erstellen, nun da die Commits schon einmal eingespielt wurden. :-(

[flyx]

Echt beschissen, wenn man plötzlich mit anderen Leuten zusammenarbeitet und nicht mehr wild die history rewriten kann :D