Skip to content

NOX: Forgot counter initialization in updated PrePostOperator test

James Willenbring requested to merge atoth1:nox-ppo-update into develop

Created by: atoth1

@trilinos/nox @rppawlo @bartlettroscoe

Description

Fixes new failure on the dashboard: https://testing-vm.sandia.gov/cdash/testDetails.php?test=44264658&build=3369951 This was due to changes in Pull Request #2377. Forgot to initialize some new counters to zero, and this wasn't getting picked up in either my testing or the pre-merge auto testing

Motivation and Context

Related Issues

  • Closes
  • Blocks
  • Is blocked by
  • Follows
  • Precedes
  • Related to
  • Part of
  • Composed of

Checklist

  • My commit messages mention the appropriate GitHub issue numbers.
  • My code follows the code style of the affected package(s).
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the code contribution guidelines for this project.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • No new compiler warnings were introduced.
  • These changes break backwards compatibility.

Additional Information

Merge request reports