Distributed CP concept along with the `single` CP allocations makes debugging complicated

[jazeee]

I was looking into the singles_CP and the oddities related to changing CP allocation. In some cases, it seems buggy to go from 1 to 2 and see CP usage drop by 2. Example scenario:

• I have 25 CP, and therefore 2 TP, which allows for 2 "singles"
• I allocate 1 skill at level 4 - for 25 CP
• I have no available CP now. (0/25) in the UI.
• I can still allocate two skills each with 1CP usage due to distributed_CP concept.

I then get 1 more CP, making me have 26CP

• I change one of the 1CP skills to a 2CP skill, but now become overallocated at 27CP.

While it works, it adds some oddities to the UI, and a bit of hidden behaviors. It is not easy to know where the 1's are being used up, either.

I wonder if it would be simpler to just give everyone 6 CP, and remove the singles concept.

[johnlamping]

I agree, but I think a different fix might work better.

The distributed_CP computation is intended to handle breadth points, without the player having to worry about them. I thought doing it automatically would be a favor to players. But it can be confusing, as you point out, and as I have noticed sometimes myself.

Maybe a better way would be to make breadth points explicit in the character sheet. There would be a BP available/used summary at the top. right after the TP summary. And "B" would be an additional option for your expertise.

You could make a decent argument that it is better to get rid of breadth points, entirely. But I kind of like the idea of characters adding additional random skills as they advance.

On Fri, Dec 30, 2022 at 6:24 PM Jaz @.***> wrote:

Assigned #41 https://github.com/epic-power-rpg/roll20/issues/41 to @johnlamping https://github.com/johnlamping.

— Reply to this email directly, view it on GitHub https://github.com/epic-power-rpg/roll20/issues/41#event-8132147358, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACTV55UT4X53CNQIX4RGY7DWP6KPFANCNFSM6AAAAAATNJLCA4 . You are receiving this because you were assigned.Message ID: @.***>

[jazeee]

The B option sounds good to me!

On Fri, Dec 30, 2022, 6:57 PM John Lamping @.***> wrote:

I agree, but I think a different fix might work better.

The distributed_CP computation is intended to handle breadth points, without the player having to worry about them. I thought doing it automatically would be a favor to players. But it can be confusing, as you point out, and as I have noticed sometimes myself.

Maybe a better way would be to make breadth points explicit in the character sheet. There would be a BP available/used summary at the top. right after the TP summary. And "B" would be an additional option for your expertise.

You could make a decent argument that it is better to get rid of breadth points, entirely. But I kind of like the idea of characters adding additional random skills as they advance.

On Fri, Dec 30, 2022 at 6:24 PM Jaz @.***> wrote:

Assigned #41 https://github.com/epic-power-rpg/roll20/issues/41 to @johnlamping https://github.com/johnlamping.

— Reply to this email directly, view it on GitHub https://github.com/epic-power-rpg/roll20/issues/41#event-8132147358, or unsubscribe < https://github.com/notifications/unsubscribe-auth/ACTV55UT4X53CNQIX4RGY7DWP6KPFANCNFSM6AAAAAATNJLCA4

. You are receiving this because you were assigned.Message ID: @.***>

— Reply to this email directly, view it on GitHub https://github.com/epic-power-rpg/roll20/issues/41#issuecomment-1368153766, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABK6O2DOAQM2SUGKUZK4YE3WP6OK3ANCNFSM6AAAAAATNJLCA4 . You are receiving this because you authored the thread.Message ID: @.***>