veevelder / quick-combat

FoundryVTT module for small Combat Enhancements
MIT License
6 stars 5 forks source link

[ISSUE] Initiative auto roll npcs doesnt work on pf2 correctly #21

Closed ThingsCouldGetDicey closed 1 year ago

ThingsCouldGetDicey commented 2 years ago

Describe the bug pf2 has an auto roll of npcs hook by shift clicking the npcs in the combat tracker. currently when using the feature to auto roll npcs in the system will pull up roll dialogues for each npc forcing the user to click all of the manually.

To Reproduce Steps to reproduce the behavior:

  1. enable npc auto roll within a pf2 system game
  2. have alot of npcs
  3. start combat
  4. See all of the dialogue rolls.

Expected behavior what i expect to replace the current behaviour is just to auto roll the intiative without the dialogue with the assumed perception skill.

Screenshots If applicable, add screenshots to help explain your problem.

Version Information

Additional context Add any other context about the problem here.

veevelder commented 2 years ago

ive added a new setting for quick combat (https://github.com/veevelder/quick-combat#pathfinder-second-edition---skip-dialog) that should take care of what you want. Let me know if there is anything else you would like to add for PF2e

ThingsCouldGetDicey commented 2 years ago

i mean if you can do it we are looking for a solution to being able to select the kind of intiative before combat starts. so a selection box of the skill checks for intiative would be amazing.

ThingsCouldGetDicey commented 2 years ago

https://github.com/foundryvtt/pf2e/issues/4267

veevelder commented 2 years ago

I can look into it, but I just want to clarify a few things as im not to familiar with PF2e.

Is the type of initiative the same for all PCs/NPCs, or can they be different? Is this something a player should determine or just the GM?

If I cant find a "simple" solution I can monitor the issue you linked to see if they make any updates and adjust my stuff as needed.

ThingsCouldGetDicey commented 2 years ago

I can look into it, but I just want to clarify a few things as im not to familiar with PF2e.

Is the type of initiative the same for all PCs/NPCs, or can they be different? Is this something a player should determine or just the GM?

If I cant find a "simple" solution I can monitor the issue you linked to see if they make any updates and adjust my stuff as needed.

the gm detemines the roll. based on what the pcs or npcs where doing prior to the call for intiative. the default is perception based checks. as thats the most common the second common is stealth rolls. the idea being that if they roll higher with stealth they get to go first in initiative. there are no surprise rounds. but for example if a pc wanted to bluff his way to get closer to an npc the intiative roll would be deception instead of the usual perception skill. this also could be different per pc and per npc. depending on what they were doing prior. but the most common is perception and stealth.

which is why if you can do something to contribute to the system that did add the intaitive selection on the roll dialogue or something that could ask the gm "hey do you wanna roll X skill for all npcs" that would be a good solution for the most part.

veevelder commented 1 year ago

Ok, so I think I may have a solution to this, I haven't pushed into the code base yet but here is what it will look like.

New setting to Group NPC rolls by initiative type

group_npc_setting

When enabled it will pop up a new window to select which skill to roll with

group_npc_dialog

This will enable all NPCs to use that skill to roll for initiative but still ask for the PCs (the window will still pop up).

group_npc_roll_result (the 1 actor is the PC)

Obviously with this option disabled it will still popup the windows for every combatant. After you've selected the skill to roll with normal quick-combat stuff will happen. So does this work for your needs or do you have something else in mind?

ThingsCouldGetDicey commented 1 year ago

usually it will be either perception as default or stealth for npcs. for players its more vital to have the selection. for that pop up. it will be much rarer for npcs to need other checks. but for players its more frequent. if they could click the intiative roll button on the combat tracker and for that drop down menu to happen for them that would be very useful to players.

veevelder commented 1 year ago

Ok, how does this look then? (GM on left player on right)

group_npc_setting

The GM initiates combat a pop up will display for him asking for the type group_roll_gm

It will then ask the players for the type group_roll_player

and once all players have submitted it will roll and start combat group_npc_roll_result

ThingsCouldGetDicey commented 1 year ago

this looks amazing. have you shared this with the pf2 devs?

ThingsCouldGetDicey commented 1 year ago

could you post a gif of this working so i could see it in action?

veevelder commented 1 year ago

Here is a gif of it working, sorry for the terrible quality.

https://media.giphy.com/media/le4xPKGehEvwLorWb2/giphy.gif

ThingsCouldGetDicey commented 1 year ago

cool i would have an option to enable gms to individually select each npc for a box selection too and with a short cut to bypass group selection when grouping them just incase. as this may come up in the future. then i think its sorted.

veevelder commented 1 year ago

ok how does this look then? New Settings Option to choose from

group_npc_setting

Default - system default it will prompt for every combatant pf2e_default

Fast - will assume Perception for every combatant, and wont prompt pf2e_fast

Prompt - will prompt GM and Players to pick which initiative type to roll with

ThingsCouldGetDicey commented 1 year ago

that is amazing! better than i expected. maybe one more option fast for gm (i.e always rolling perception) prompt for players? but what youve done for prompt is how id use it.

so the way i understand it you get the prompt at the top to group npcs to whatever that box is but if you wanted to individualise it you could by unselecting it and doing it manually right?

veevelder commented 1 year ago

I can add that option.

and yeah, if you have that checkbox selected it will roll whatever you have selected in the drop down for every NPC.

however if you uncheck that checkbox, for each NPC listed below it will roll whatever you have in their drop downs

ThingsCouldGetDicey commented 1 year ago

also intaitive rolls shouldnt default to blind. they should be public rolls. just to be clear.

ThingsCouldGetDicey commented 1 year ago

there maybe an instance where the gm may want to roll in secret. so you could put in a clause when the gm rolls stealth for inatiative. its a non public roll. not even blind because the gm wouldnt want the players to see the dice. then all bases are covered.

veevelder commented 1 year ago

I don't think the pf2e system will let me set the chat output options, but if the GM sets the roll type before rolling it will be set for all the rolls, so they can use it to set public/private rolls in that case. I can set things as secret but that doesn't appear to be the same thing.

without secret public rolls public_rolls

with secret rolls public rolls public_rolls_secret

players view secret_rolls_player

im inclined to leave it as is with the secret rolls enabled and the GM can reveal them whenever they want to. In reality it doesnt make that much of a difference when looking at the turn tracker (except for hidden tokens). And adding more buttons to things that should be automated feels like taking a step backwards.

ThingsCouldGetDicey commented 1 year ago

fair enough just wanted to make sure all bases are covered. you have done a fantastic job.

veevelder commented 1 year ago

the module has been updated, let me know if you find an issues with it