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

Correction of Hybrid Model Reporting Issues #10581

Merged
merged 3 commits into from
Jun 25, 2024

Commits on Jun 21, 2024

  1. Fix for Defect #10508

    Fixes the defect with a solution suggested in GitHub which includes the following:
    1. Calculates average multiplier regardless of system operation (now simply calculates it every time)
    2. Uses the limit of 30 as a warning threshold but does not eliminate the value (previously set to 1 so that it wouldn't affect the average)
    These items have been accomplished through the creation of a new subroutine that controls the calculation limits and calculates the sum, count, and average.
    RKStrand committed Jun 21, 2024
    5 Configuration menu
    Copy the full SHA
    9ef7d61 View commit details
    Browse the repository at this point in the history
  2. Additions, Unit Test, Docs for Defect #10508

    This commit includes:
    1. Addition of a new report variable for the output parameter that spurred this defect.
    2. A unit test was added to exercise the new subroutine through its various cases.
    3. Documentation edits for both the input-output and engineering references.
    Potential PR candidate.
    RKStrand committed Jun 21, 2024
    Configuration menu
    Copy the full SHA
    1ca21b8 View commit details
    Browse the repository at this point in the history
  3. 5 Configuration menu
    Copy the full SHA
    320219f View commit details
    Browse the repository at this point in the history