Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Calculation of "Grundwerte" uses base attribute values only #1

Closed
YuanRamirez opened this issue Mar 6, 2017 · 6 comments
Closed

Calculation of "Grundwerte" uses base attribute values only #1

YuanRamirez opened this issue Mar 6, 2017 · 6 comments
Assignees

Comments

@YuanRamirez
Copy link

Lifepoints, Stamina and Astral Energy are calculated with base attributes only (should be base + modifiers)

@kreuvf kreuvf self-assigned this Mar 6, 2017
@kreuvf
Copy link
Owner

kreuvf commented Mar 6, 2017

(If I should write in English, please say so.)

Ich denke, dass das Problem hier ist, dass der Heldenbogen als Modifikator nicht den Modifikator aus der Generierung meint, sondern temporäre Veränderungen durch einen Attributo oder Krankheiten/Gifte. Die Grundwerte werden in solchen Fällen nämlich nicht neu ausgerechnet - AT/PA/FK-Basiswerte jedoch schon, weshalb das bei diesen so eingebaut ist.

Hinweis: Der Bogen ist größtenteils nicht von mir, ich habe ihn nur entdeckt und versuche ihn so gut es geht zu verbessern. Das von dir beschriebene Problem bestand schon vor meinen Änderungen, weshalb ich mir nicht absolut sicher sein kann, dass meine Interpretation stimmt.

Eine Lösung für dieses Problem sollte aber schnell gefunden sein: Entweder eine bessere Benennung der Spalten (inkl. Ausfüllhinweisen) oder Hinzufügen eines zusätzlichen Tabs für die Werte aus der Generierung (mein Favorit ^.^). Durch einen solchen Tab könnte das Character Sheet auch erkennen, wenn eine Eigenschaft über den Maximalwert hinaus geändert werden soll.

@kreuvf kreuvf added the dsa4.1 label Mar 6, 2017
@kreuvf kreuvf changed the title Das Schwarze Auge 4.1 - Calculation of "Grundwerte" uses base attribute values only Calculation of "Grundwerte" uses base attribute values only Mar 6, 2017
@YuanRamirez
Copy link
Author

Ah, achso! Dann hab ich das einfach nicht richtig gecheckt ^^'
Klingt aber soweit ganz gut. Ein zusätzlicher Tab ist vermutlich deutlich mehr Arbeit, oder nicht? Ich meine, wenn das nicht zu viele Umstände sind, ist es die Mehr-Arbeit wahrscheinlich wert, einfach um auch in Zukunft weitere Nachfragen wie die meine zu verhindern :D

@kreuvf
Copy link
Owner

kreuvf commented Mar 8, 2017

Ein zusätzlicher Tab ist nicht viel Arbeit. Die Schwierigkeit wird nur darin bestehen dieses neue Feature so einzubauen, dass es automatisch mit den Daten aus alten Bögen funktioniert. Übrigens: Dass du das nicht richtig gecheckt hast, mag vielleicht zutreffen, ist aber ein Zeichen für schlechtes Design des Benutzerinterfaces. ^^ Aufgrund mangelnder Dokumentation ist auch meine Interpretation nicht mehr als bloßes Raten.

@kreuvf
Copy link
Owner

kreuvf commented Mar 12, 2017

So, habe nochmal die Regelwerke abgesucht und auch mit meinem Meister Rücksprache gehalten: Die Grundwerte berechnen sich nicht neu, wenn sich Eigenschaften temporär verändern. Nicht umsonst steht bei Wunden dazu, wenn sich AT/PA verändern.

Daher werde ich das so umbauen, dass auch MR, INI, AT, PA und FK sich nicht mehr auf Basis temporärer Veränderungen verändern. Ich überlege aber noch, ob es tatsächlich ein eigenes Generierungstab braucht ;D

kreuvf pushed a commit that referenced this issue Mar 22, 2017
kreuvf pushed a commit that referenced this issue Mar 22, 2017
kreuvf pushed a commit that referenced this issue Apr 22, 2017
kreuvf pushed a commit that referenced this issue Apr 22, 2017
kreuvf pushed a commit that referenced this issue Jun 5, 2017
kreuvf pushed a commit that referenced this issue Jun 20, 2017
kreuvf pushed a commit that referenced this issue Jul 15, 2017
kreuvf pushed a commit that referenced this issue Jul 15, 2017
Cwod cleanup: Put Mental attributes in proper order
kreuvf pushed a commit that referenced this issue Jul 17, 2017
Eirendor (spanish rpg) Character Sheet added
kreuvf pushed a commit that referenced this issue Jul 17, 2017
@Markus1204
Copy link

Ich schreib das hier mal rein weil es hier auch um die Grundwerte geht... Manchmal - und mir will nicht in den Kopf warum das so ist - passiert folgendes:

Ich trag die Grundwerte in den Sheet ein indem ich ganz normal die Basis verändere, denn Modis sind nur für temporäre Mali / Boni etc. gedacht... Die Werte werden auch erstmal übernommen und alles ist richtig.

Wenn ich dann aber während einer anderen Sitzung den Sheet öffne kann es sein, dass zwar die veränderten Basis Werte noch stimmen, aber die Endwerte (Basis + Mod) einfach mal ein paar Punkte darunter liegen... Es sind KEINE Modifikatoren eingetragen und auch die Basis wurde nicht weiter verändert...

Das einzige was bisher geholfen hat war den besagten Sheet zu löschen und für diesen Charakter einen neuen Sheet anzulegen inklusive dem erneuten Übertragen... Es kommt zwar nur ab und an vor, allerdings hab ich keine Ahnung was diesen Bug auslöst...

kreuvf pushed a commit that referenced this issue Aug 24, 2017
kreuvf pushed a commit that referenced this issue Sep 6, 2017
@kreuvf
Copy link
Owner

kreuvf commented Sep 16, 2017

Solange ich den Bug nicht selbst reproduzieren kann, kann ich ihn leider auch nicht beheben. Wenn du das reproduzieren kannst (und nur dann :P), verdient der Bug ein eigenes Issue. Sollte ich dieses Issue also schließen, bedeutet das nicht automatisch auch, dass dein Bug behoben ist ^.^

kreuvf pushed a commit that referenced this issue Oct 8, 2017
kreuvf pushed a commit that referenced this issue Oct 8, 2017
Change to reflect account ownership.
kreuvf pushed a commit that referenced this issue Oct 13, 2017
kreuvf pushed a commit that referenced this issue Oct 13, 2017
kreuvf pushed a commit that referenced this issue Nov 28, 2017
kreuvf pushed a commit that referenced this issue Nov 25, 2020
Updating Repository to latest version
kreuvf pushed a commit that referenced this issue Nov 25, 2020
kreuvf pushed a commit that referenced this issue Nov 25, 2020
kreuvf pushed a commit that referenced this issue Nov 25, 2020
kreuvf pushed a commit that referenced this issue Nov 25, 2020
Update fork from source
kreuvf pushed a commit that referenced this issue Nov 25, 2020
kreuvf pushed a commit that referenced this issue Apr 8, 2021
Merge my:master with Roll20:Master
kreuvf pushed a commit that referenced this issue Apr 8, 2021
kreuvf pushed a commit that referenced this issue Apr 8, 2021
kreuvf pushed a commit that referenced this issue Apr 8, 2021
Sync with roll20 master
kreuvf pushed a commit that referenced this issue Apr 8, 2021
kreuvf pushed a commit that referenced this issue Jul 30, 2021
Update from the orginal roll20
kreuvf pushed a commit that referenced this issue Jul 30, 2021
kreuvf pushed a commit that referenced this issue Sep 11, 2021
kreuvf pushed a commit that referenced this issue Sep 11, 2021
* Add movement penalty to armor

* Add handler on spd

* D`repeating_armor_${id}_movementpenalty`:wal with active armor
kreuvf pushed a commit that referenced this issue Oct 27, 2021
kreuvf pushed a commit that referenced this issue Dec 24, 2021
kreuvf pushed a commit that referenced this issue Jan 30, 2022
kreuvf pushed a commit that referenced this issue Feb 7, 2022
kreuvf pushed a commit that referenced this issue Feb 20, 2022
kreuvf pushed a commit that referenced this issue Sep 23, 2022
kreuvf pushed a commit that referenced this issue Sep 23, 2022
kreuvf pushed a commit that referenced this issue May 24, 2023
kreuvf pushed a commit that referenced this issue Jul 22, 2023
[pull] master from Roll20:master
kreuvf pushed a commit that referenced this issue Jan 22, 2024
Changing CRLF to LF and change lowercase j to uppercase J.
kreuvf pushed a commit that referenced this issue Mar 24, 2024
…2024

Chasing Adventure - Several New Features
kreuvf pushed a commit that referenced this issue Apr 14, 2024
This is a continuation of the beta test for the Unofficial HK RPG I've been working on over these last 2 months. The first page is now 99% complete. Additional pages will be implemented in coming updates.
kreuvf pushed a commit that referenced this issue Apr 14, 2024
…ght-RPG

The Unofficial Hollow Knight RPG (Beta Update #1, bugfixes and new features)
kreuvf pushed a commit that referenced this issue Jul 28, 2024
Torchbearer 2E: fix trait text and remove background
kreuvf pushed a commit that referenced this issue Sep 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants