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.

Sign up for GitHub

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

Jump to bottom

Improved StratConSkillGenerator #6021

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Improved StratConSkillGenerator #6021

wants to merge 2 commits into from
+20 −25

Commits on Sep 20, 2024

  1. Refactor StratConSkillGenerator skill logic 

    Refactored the skill generation logic in StratConSkillGenerator to streamline skill level calculations and removed redundant bonus handling. Also cleaned up logging in AtBDynamicScenarioFactory and removed an unnecessary skill generator type setting.
    @IllianiCBT
    IllianiCBT committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    3ba9723 View commit details
    Browse the repository at this point in the history

  2. Add random roll to skill level adjustment 

    A random dice roll was introduced to adjust the skill level in the StratConSkillGenerator class. A roll of 1 decreases the skill level while a roll of 6 increases it. This change aims to introduce more variability to skill level determination.
    @IllianiCBT
    IllianiCBT committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    f076d31 View commit details
    Browse the repository at this point in the history