-
-
Notifications
You must be signed in to change notification settings - Fork 19
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
[BUG] Critical Damage Application With Multiple Rolls Enabled #276
Comments
@TLad75 the issue with maximizing dice on the right is that dnd5e now supports adding additional melee damage dice and damage (e.g. for brutal critical), which shouldn't be maximized by powerful critical. The intended behaviour of powerful critical is always to maximize the original dice (i.e. the left hand side). The issue is, both left hand side and right hand side rolls are one roll each. I cannot maximize part of a roll, so the only options I have are to maximize the left hand side (which gives the correct powerful critical result), or to maximize the right hand side entirely, bonus damage included. Neither option is ideal of course. I will attempt to investigate a potential way to mimic maximizing internally, thereby hopefully taking the maximizing back to the right hand side and maximizing the roll partially (excluding bonus dice), but I expect this will be quite tricky. |
Ah thank you for the detailed explanation, that makes sense. I'm guessing I was looking at an older version of the mod since that stream was from about a year ago which was on V10 of Foundry based on release notes. If it is possible to mimic maximizing internally that would be an awesome addition to multiple rolls to streamline combat. Since it is not how the module works currently, feel free to close this bug out if you would like or move it to a feature request. I will get my players into the habit of rolling advantage/disadvantage with shift and ctrl which should hopefully be easy to do since they'll be learning Foundry from the ground up. Thanks again! |
@TLad75 also remember there's retroactive adv/disadv buttons on chat cards as well, if your players forget to roll with advantage or disadvantage pre-emptively. Multiple rolls aren't really needed in most case, they're sort of a legacy feature from Better Rolls, but some people prefer them. I'll keep this ticket open to track this issue, but moving it to feature. |
As of version 3.0.0 (which works with dnd5e 3.0), RSR no longer handles the internal logic of calculating crits and instead allows the system to do so internally. As such, this ticket is no longer related to RSR and can be considered resolved from the standpoint of this module. |
Description
First off, I just started moving to Foundry and wow I love this mod, thank you! I am not entirely sure if this is a bug, intended behavior or if I am doing something wrong. I have 'Always Roll Multiple Dice' enabled in RSR as well as 'Critical Damage Maximize Dice' enabled in the DnD5e core settings. When rolling attack rolls and if either D20 is a 20 it will calculate critical damage as expected by maxing the base damage, add the ability modifier and then do a roll for the critical damage on top of that. This can cause some confusion when multiple rolls are enabled and the right hand roll is the nat 20 (right hand roll is used for advantage when the situation calls for it).
I have seen in a couple issues here (120 and specifically 262) that when these settings are applied then this maximized critical damage should apply to the additional damage value (on the right side) instead which would make determining the normal damage when the attack is not actually a crit much easier as the rolled damage would have the modifiers added and the it would be + Maxed damage dice Crit on the right side.
This is from a D&D stream that aired roughly a year ago but this is what I am attempting to reproduce (4d8 + 9 on the left then the maxed 4d8 on the right):
Steps to Reproduce
Versions
Other Modules
libWrapper 1.12.13.0
The text was updated successfully, but these errors were encountered: