crash1115 / 5e-training

A module for Foundry VTT that allows users to keep track of downtime activities, quest progress and... Well, pretty much anything you can track with a loading bar and a %.
MIT License
13 stars 12 forks source link

Armor Class calculation error. #73

Closed Forget779 closed 1 year ago

Forget779 commented 2 years ago

I have a barbarian in my party as I was creating their character using this module and the armor class formula for unarmored defense via barbarian, the AC remained the default ten on the sheet. The attached image shows the character sheet of the half-orc barbarian, and the problematic calculation display. OGL_CS_Error

Forget779 commented 2 years ago

As you can see the proper math displays on the configuration settings but not on the character sheet itself. Additionally, it would only update the AC without armor if and only if the dexterity mod is greater than or less than 0 not effected by the constitution score.

crash1115 commented 2 years ago

Hey there, thanks for the report!

Seems like you've got a module installed that's adjusting the style of the character sheet - does this issue still occur if you set the sheet to use the vanilla dnd5e character sheet?

Forget779 commented 2 years ago

I will attempt this tomorrow. To try and assess the difference I am using the OGL character sheet module where the glitch is occurring initially. I will double check soon.

On Thu, Oct 28, 2021 at 10:23 crash1115 @.***> wrote:

Hey there, thanks for the report!

Seems like you've got a module installed that's adjusting the style of the character sheet - does this issue still occur if you set the sheet to use the vanilla dnd5e character sheet?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/crash1115/5e-training/issues/73#issuecomment-953897809, or unsubscribe https://github.com/notifications/unsubscribe-auth/AWDPX6NCTRLC4W2FC5QU5STUJFMEPANCNFSM5G2M6S2Q . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

crash1115 commented 2 years ago

Sounds good, thanks. I'm just trying to narrow down if this is an issue with the sheet module or not; mine doesn't really touch anything that should mess with the AC calculation, so knowing if it's sheet specific can help me narrow down what the issue might be.

Forget779 commented 2 years ago

Unfortunately, I have tried it with the vanilla and the OGL module. The obsidian module allows the DM to adjust the armor class manually but not by a preset feature. I prefer the OGL module sheet because I can have the party track the quests they take. However, this module is very useful.

On Thu, Oct 28, 2021 at 12:57 PM crash1115 @.***> wrote:

Sounds good, thanks. I'm just trying to narrow down if this is an issue with the sheet module or not; mine doesn't really touch anything that should mess with the AC calculation, so knowing if it's sheet specific can help me narrow down what the issue might be.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/crash1115/5e-training/issues/73#issuecomment-954029452, or unsubscribe https://github.com/notifications/unsubscribe-auth/AWDPX6JB4XYTASEXDFTTSYDUJF6IFANCNFSM5G2M6S2Q . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

crash1115 commented 2 years ago

Interesting. I can't seem to reproduce this issue, so it might be another module conflict, something strange with the particular actor, or something I'm just missing entirely. Couple more questions:

  1. What versions of foundry and dnd5e are you running?

  2. If you open the character sheet then open the console (F12 for windows > console tab), do you see any errors (red text)?

  3. Can you export the actor you're having trouble with (right click it in the actors sidebar > export) and attach it to a comment in this thread? You'll probably have to put the json file into a zip folder, then drag/drop that into a reply. Should do the trick.

Thanks!

Forget779 commented 2 years ago

Will do. I’ll have that into you soon. On Fri, Oct 29, 2021 at 15:36 crash1115 @.***> wrote:

Interesting. I can't seem to reproduce this issue, so it might be another module conflict, something strange with the particular actor, or something I'm just missing entirely. Couple more questions:

1.

What versions of foundry and dnd5e are you running? 2.

If you open the character sheet then open the console (F12 for windows

console tab), do you see any errors (red text)? 3.

Can you export the actor you're having trouble with (right click it in the actors sidebar > export) and attach it to a comment in this thread? You'll probably have to put the json file into a zip folder, then drag/drop that into a reply. Should do the trick.

Thanks!

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/crash1115/5e-training/issues/73#issuecomment-955001668, or unsubscribe https://github.com/notifications/unsubscribe-auth/AWDPX6M4MGR3RP7W4YJIMGDUJLZUHANCNFSM5G2M6S2Q . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.