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

The collapsible legend is wrong #156

Open
NoudL opened this issue Nov 5, 2018 · 2 comments
Open

The collapsible legend is wrong #156

NoudL opened this issue Nov 5, 2018 · 2 comments
Labels

Comments

@NoudL
Copy link

NoudL commented Nov 5, 2018

The collapsible legend in the lower right corner gives information such as
"Group: Four of these is a medium challenge"
it says that about the light purple color which corresponds to 1/4 CR when using a party of 4 level 1 characters. However, building an encounter of four 1/4CR enemies is actually "Deadly", which the encounter builder correctly states.
I think the legend doesn't take the Adjusted XP into mind. Because 1/4CR creatures give 50 xp. 50 * 4 is 200 which would correspond to a "medium" challenge were it not for the adjusted xp rule.

@Asmor
Copy link
Owner

Asmor commented Nov 5, 2018

It's an estimation which isn't perfect. I haven't looked at the code for a long time, but it seems like the estimation is less accurate at lower levels.

I just checked with a party of 4 level 10 characters, and CR 3 is marked as pink, and 4 of them is indeed a "medium" challenge per the encounter building guidelines.

@sbards
Copy link

sbards commented Dec 2, 2018

With a party of 4 level 10 characters, a CR 4 is marked as pink as well, however 4 of them are a hard challenge, not medium.

4 level 10 characters:
Easy: 2,400 exp
Medium: 4,800 exp
Hard: 7,600 exp
Deadly: 11,200 exp

4 CR 4 creatures:
Total XP: 4,400 (1,100 per player)
Adjusted XP: 8,800 (2,200 per player)
Difficulty: Hard
Legend Color: Pink "Group: Four of these is a medium challenge"

Looks like the legend is reading from the total xp, not the adjusted xp. I'll see if I can't fix it and get something submitted once I'm more familiar with the code base.

@Asmor Asmor added the bug label Dec 29, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants