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

add tests #958

Merged
merged 39 commits into from
Nov 12, 2024
Merged

add tests #958

merged 39 commits into from
Nov 12, 2024

Conversation

a-zakir
Copy link
Contributor

@a-zakir a-zakir commented Oct 28, 2024

The pull request introduces several changes to the project, including:

  1. Updating the build scripts for various platforms (CentOS 7, Oracle 8, Ubuntu, and Windows) to use a new Cucumber test feature for Benders Outputs.

  2. Modifying the C++ code for the Benders Math Logger to include delimiters in the output files. This change ensures that the data in the output files is properly formatted and easy to parse.

  3. Adding new C++ tests for the Benders Math Logger to verify that the data is being written to the output files correctly.

  4. Updating the Cucumber test framework to include a new feature for testing Benders criterion outputs. This feature includes steps for running the simulation, checking the simulation time, and verifying the expected positive unsupplied energy and loss of load.

  5. Adding a new example test for the SmallTestFiveCandidatesWithWeights study, which includes steps for running the simulation and checking the expected overall cost, investment cost, and solution.

  6. Updating the utils_functions.py file to include a new function for reading outputs from the simulation, which can handle both archived and non-archived output files.

Overall, these changes improves the testing and logging capabilities of the project, as well as adding new functionality for handling Benders criterion outputs.

@pull-request-size pull-request-size bot added size/L and removed size/S labels Oct 29, 2024
@pull-request-size pull-request-size bot added size/XL and removed size/L labels Nov 6, 2024
@pull-request-size pull-request-size bot added size/L and removed size/XL labels Nov 6, 2024
@pull-request-size pull-request-size bot added size/XL and removed size/L labels Nov 6, 2024
@a-zakir a-zakir marked this pull request as ready for review November 8, 2024 11:59
@a-zakir a-zakir merged commit 6a3ba09 into develop Nov 12, 2024
25 checks passed
@a-zakir a-zakir deleted the feature/test-new-outputs branch November 12, 2024 09:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants