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

AVR-720 When both advantage and disadvantage exist on the same armor, stealth checks roll with advantage instead of cancelling #1431

Open
damccull opened this issue Jan 31, 2021 · 9 comments
Labels
bug ddb This issue's root cause is upstream in DDB P4: Low Issues that may be implemented eventually. Scope: Few This issue affects only a few people. Severity: Minor This issue doesn't have much of an impact.
Projects

Comments

@damccull
Copy link

Describe the bug
When both advantage and disadvantage exist on the same armor, stealth checks roll with advantage instead of cancelling

Severity
Medium (minor functional impact)

Context
!check stealth

To Reproduce
Import DnDBeyond Artificer Armorer subclass with Scale Mail armor (medium armor)
Be level 3, and choose Infiltrator class Armor model
See both advantage and disadvantage on Stealth skill
Import character to avrae
!check stealth

Expected behavior
A normal roll without advantage or disadvantage

Screenshots
image
image

Additional context
Add any other context about the problem here.

@damccull damccull added the bug label Jan 31, 2021
@taine-bot
Copy link
Collaborator

Tracked as AVR-720.

@taine-bot taine-bot changed the title When both advantage and disadvantage exist on the same armor, stealth checks roll with advantage instead of cancelling AVR-720 When both advantage and disadvantage exist on the same armor, stealth checks roll with advantage instead of cancelling Jan 31, 2021
@1drturtle
Copy link
Contributor

Hi! Thanks for reporting a bug! If you could hop on the Avrae Dev discord server, we can try and help you out. In the future, it would be appreciated if you came to the server first so we can try to debug the issue before opening a bug report!

Link to the support server: https://support.avrae.io

@zhudotexe zhudotexe added this to Needs triage in Bugs Feb 3, 2021
@zhudotexe zhudotexe added Scope: Few This issue affects only a few people. Severity: Minor This issue doesn't have much of an impact. labels Feb 3, 2021
@zhudotexe
Copy link
Collaborator

Hi there - please attach a link to the affected character to help reproduce this issue

@damccull
Copy link
Author

damccull commented Feb 8, 2021

@zhudotexe
Copy link
Collaborator

zhudotexe commented Feb 17, 2021

The root cause of this is that the disadvantage from wearing armor has a restriction on it in the data, "with armor worn" - obviously this is true, but there's no way to programatically determine this, and Avrae ignores conditional adv/dis (otherwise e.g. Drow would have permanent dis on Perception). The advantage from Armor Model has no such condition, so only the advantage is considered.

@zhudotexe
Copy link
Collaborator

Screenshot 2021-02-17 at 14 11 20

@damccull
Copy link
Author

damccull commented Feb 17, 2021

Ok, that's interesting. Looks like I can append 'dis' to the !check command to cancel it out manually. That can work as a workaround. Unsure if this should be fixed or not. You may close if not.

@zhudotexe zhudotexe added the ddb This issue's root cause is upstream in DDB label Feb 17, 2021
@zhudotexe zhudotexe moved this from Needs triage to Medium priority in Bugs Feb 17, 2021
@Haravikk
Copy link

Haravikk commented Apr 1, 2021

I was going to post about the lack of auto-disadvantage as an issue, but this one seems similar enough.

The correct "fix" to me would seem to be to impose disadvantage whenever armour that imposes it is worn, as I expect most players will assume their armour is worn at all times. The only issue is how easy it is to remove the armour; with a DDB linked account it's fairly easy to unequip it then !update to reflect the change, but is a command needed to make it possible to do it directly from Avrae?

I'm constantly having to remind players to roll with disadvantage because of their heavy armour, but it seems like auto-disadvantage should be common enough to be the default with those types of armour.

Only other issue I can think of would be whether Avrae can handle mithril armour?

@stale
Copy link

stale bot commented Jun 9, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs.

@stale stale bot added the stale Issues that have no activity for 30 days. label Jun 9, 2021
@zhudotexe zhudotexe added the P4: Low Issues that may be implemented eventually. label Jun 9, 2021
@stale stale bot removed the stale Issues that have no activity for 30 days. label Jun 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug ddb This issue's root cause is upstream in DDB P4: Low Issues that may be implemented eventually. Scope: Few This issue affects only a few people. Severity: Minor This issue doesn't have much of an impact.
Projects
Bugs
Medium priority
Development

No branches or pull requests

5 participants