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

Change to Power Points, Added Bestiary, and Minor Fixes #117

Merged
merged 19 commits into from
Aug 20, 2022

Conversation

LockerM
Copy link
Collaborator

@LockerM LockerM commented Aug 18, 2022

Wanted to add this PR with fewer Issues resolved so we can close, merge, and then work to address the MISSING Issues.

In this PR:

I will hold off on adding any additional commits not related to the above issues so we can close/merge more quickly.

Created Bestiary + Template in YAML using @CBroz1 submitted template.
Updated Language on Epic Event determining which skill to use (line 84 PGB)
Corrected mispelling of Erdania.
Caster name change
Defender name change

Indicating only first 3 levels are covered.
Added "Creating your Dealer" section

Minor fix for Numbers separated by a `-` instead of `/`

Fixed broken link.
Converted *FP* and *Fate Points* to *PP* and *Power Points*.
Removed from .yml (not .md yet).
Updated Changelog to resolve and timestamp a3 and a4, with 1.0.1 marked as UNRELEASED.
@LockerM LockerM added this to the 1.0.0a4 milestone Aug 18, 2022
@LockerM LockerM requested a review from CBroz1 August 18, 2022 13:50
Minor in-line edits on Player Guide Full from @D-D-2's fork.
Removed TheAdventures specific Dealers and some terminology/elements from initial YAML v1 Bestiary.
@LockerM LockerM marked this pull request as draft August 18, 2022 14:10
Removed Dealers, added notes on default Range and to hit.

Added additional Actions to enemies and streamlined naming conventions to make Setting agnostic. This approach will give us a template to work off of for creatures of various power and difficulty without needing to bloat an initial Bestiary.

Will tackle Dealers in a separate pass.
@LockerM LockerM marked this pull request as ready for review August 18, 2022 20:10
Copy link
Collaborator

@CBroz1 CBroz1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please push a commit that hard wraps new additions.

Let's find 10min to discuss beastiary yaml structure before merging that in. Currently, they include powers that are structured differently than the full powers list. Which should we use? The segmented approach currently the the beastiary will help it be more machine readable, but to what end, what goals to we have in mind? The current approach in the powers yaml is more human readable. I would need to write code to revise how we rendered powers for player sheets if we took the segmented approach.

I think we should take the same approach for both so that the beastiary draws from the powers list when rendering (even if some powers are limited to certain creature types, players, companions, etc).

3_Automation/06_Bestiary.yml Outdated Show resolved Hide resolved
3_Automation/06_Bestiary.yml Outdated Show resolved Hide resolved
3_Automation/06_Bestiary.yml Outdated Show resolved Hide resolved
3_Automation/06_Bestiary.yml Outdated Show resolved Hide resolved
3_Automation/06_Bestiary.yml Outdated Show resolved Hide resolved
1_Mechanics/03_CharacterCreation.md Outdated Show resolved Hide resolved
1_Mechanics/03_CharacterCreation.md Outdated Show resolved Hide resolved
1_Mechanics/03_CharacterCreation.md Show resolved Hide resolved
1_Mechanics/02_GMGuide.md Outdated Show resolved Hide resolved
1_Mechanics/02_GMGuide.md Show resolved Hide resolved
@LockerM
Copy link
Collaborator Author

LockerM commented Aug 19, 2022

@CBroz1 - Agreed on the YAML for the bestiary we can use the Powers list to do the heavy lifting and then just tag each Creature in the Bestiary with the Powers name to streamline edits and maintaining the list.

I can revise the Bestiary YAML file to remove all Power effects and just keep the name under Actions.

Would we add House and Adversary Actions in the Powers list or the Actions tab in the Bestiary for Dealers and Bosses?

@CBroz1
Copy link
Collaborator

CBroz1 commented Aug 19, 2022

Would we add House and Adversary Actions in the Powers list or the Actions tab in the Bestiary for Dealers and Bosses?

Sure, that makes sense to me. Do you think they can fit in the existing Powers format? Does the Powers format need to change to have the Save structure, or should we put that off and just have sentence mechanics for now?

@@ -43,45 +43,30 @@ Name: # must be unique
Type: Companion, level 1
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Treating all creatures, including PCs as the same will result in some issues differentiating Eagle level 1 from level 2. If I want to look up Eagle level 2, where would that be? Maybe Eagle 1 needs to be creature name or

Eagle:
    Level1:
        HP: 1 (and other stats)
    Level2: 
        HP: 2

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the solution here is for unique names that indicate power/strength as opposed to level.
Wolf --> Dire Wolf
Falcon --> Bladed Falcon

Or we give a stat boost on level up in certain cases without changing the name, like adding a formula to Companion HP and +to hit:

  • Companion HP = 1+ (Primary Skill Mod/2)
  • Companion to hit = 0 + Primary Skill Mod

Removed Actions/Powers from the Bestiary current version until resolution on YAML file structure.

Changed Bystander to NPC.

Removed Phases as this should be handled by an Adventure Guide not the stat block.
@LockerM
Copy link
Collaborator Author

LockerM commented Aug 20, 2022

@CBroz1 - I updated the Bestiary in this PR to only have stat blocks for now until we can resolve the YAML file structure for the Powers list and how that will merge with Character Creation and the Bestiary.

Take a look at the updated files after the hard wrap and this streamlined Bestiary for now and let's see if we can merge this so we can tackle additional MISSING items in the PGF.

@LockerM
Copy link
Collaborator Author

LockerM commented Aug 20, 2022

Also I removed Phases from the Bestiary as I think this is best handled in an Adventure Guide or on a case-by-case basis since a Necromancer fighting in their Lair during one Adventure might be very different than a Necromancer who is invading an allied Fortress and must be defended against while they throw their armies of undead at the Fortress walls.

@CBroz1
Copy link
Collaborator

CBroz1 commented Aug 20, 2022

Necromancer fighting in their Lair during one Adventure might be very different than a Necromance

This to me is an argument in favor of different 'versions' of a creature under the main heading ('level1', 'level2') or ('in lair', 'elsewhere') as opposed to renaming the creature for each different piece.

@CBroz1
Copy link
Collaborator

CBroz1 commented Aug 20, 2022

  • recent push implies a resolution for Need a "Non-Dealer" unifying term #104 in favor of NPC. Is this a user-facing term or a backend term? if so, should it change in guide docs?
  • please resolve all review comments that have been addressed

@LockerM
Copy link
Collaborator Author

LockerM commented Aug 20, 2022

@CBroz1 - * recent push implies a resolution for Need a "Non-Dealer" unifying term #104 in favor of NPC. Is this a user-facing term or a backend term? if so, should it change in guide docs?

  • I pushed the change but am not sure on the full Issue resolution as a result, see below

--

I'm not sure still as the term "NPC" feels fairly comprehensive to include all creatures (including Dealers) so I don't know that we would need to identify them in the Bestiary as NPC vs. Dealer or if we simply would indicate in the Bestiary if they're a Dealer (or if only Player PCs are called Dealers and we revert to all GM controlled characters are now NPCS).

@LockerM
Copy link
Collaborator Author

LockerM commented Aug 20, 2022

Necromancer fighting in their Lair during one Adventure might be very different than a Necromancer..

This to me is an argument in favor of different 'versions' of a creature under the main heading ('level1', 'level2') or ('in lair', 'elsewhere') as opposed to renaming the creature for each different piece.

With regards to including "Phases" - I think a creature can have one main stat block in the bestiary and then in a specific Adventure Guide it can describe any changes made to the NPC/Dealer then rather than us trying to maintain multiple variants of an NPC every time we write a new adventure that includes modifications. But most especially for phases, because the Bestiary will get bloated/confusing if we have multiple 3 phase boss fights completely listed out in the doc when a boss might be very specific to a single adventure where they have phases but otherwise are just treated as a normal enemy in other adventures where they aren’t a boss and don’t have any phases. My thought is that a GM can turn any Dealer into a Boss encounter by adding:

  • Adversary Actions
  • House Actions*
  • Phases*

*would be specific to the GM and each adventure it’s included in

This streamlines our development of the Bestiary and then allows for a GM to say "Reference the Necromancer stat block in the Bestiary, but with the following changes..." and then they can modify damage, potential Powers, Phases, and House Actions to be specific to the setting, the environment of the campaign, etc. without us needing to account for all of these variables in a single doc that will certainly grow over time as more creatures are added by us and by the community.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants