Created OptimizationProblem's new LinearInequalityConstraintMatrices and LinearEqualityConstraintMatrices

[kwesiRutledge]

Summary

• Introduced the two methods: LinearInequalityConstraintMatrices and LinearEqualityConstraintMatrices for organizing Linear Program constraints into simple terms.
• Modified the readme.md to reflect the new way that we can structure OptimizationProblem objects. Looks good!

[Copilot]

Pull Request Overview

The purpose of the PR is to introduce new methods for organizing linear optimization constraints—specifically, the LinearInequalityConstraintMatrices method (with the LinearEqualityConstraintMatrices method referenced in the title intended for future implementation).

• Added multiple tests in optimization_problem_test.go for verifying matrix dimensions from linear inequality constraints.
• Implemented the LinearInequalityConstraintMatrices method in optimization_problem.go to assemble constraint matrices and vectors.
• Updated README.md to reflect the new structure for OptimizationProblem objects.

Reviewed Changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 2 comments.

File	Description
testing/problem/optimization_problem_test.go	Added tests to cover various cases for linear inequality constraints.
problem/optimization_problem.go	Introduced the LinearInequalityConstraintMatrices method with debug logging and variable naming.
README.md	Revised code example to utilize the new OptimizationProblem structure.

Files not reviewed (1)

• go.mod: Language not supported

Comments suppressed due to low confidence (1)

problem/optimization_problem.go:366

• [nitpick] Consider using camelCase (e.g., scalarConstraints) instead of snake_case to align with Go naming conventions.

scalar_constraints := make([]symbolic.ScalarConstraint, 0)

[Copilot]

Remove extraneous debug print statements to avoid unnecessary output in production code.

Suggested change

	fmt.Printf("Variables: %v\n", x)
	// Removed extraneous debug print statement

[Copilot]

Remove or disable debug print statements (along with similar ones) before finalizing the production implementation.

Suggested change

	fmt.Printf("AOut: %v\n", AOut)
	fmt.Printf("bOut: %v\n", bOut)
	// Debug print statements removed for production.
	// Debug print statements removed for production.

[codecov]

Codecov Report

Attention: Patch coverage is 97.16312% with 4 lines in your changes missing coverage. Please review.

Project coverage is 86.76%. Comparing base (dece87d) to head (5f4c727).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
problem/optimization_problem.go	97.16%	2 Missing and 2 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main       #8      +/-   ##
==========================================
+ Coverage   86.35%   86.76%   +0.41%     
==========================================
  Files          27       27              
  Lines        3546     3687     +141     
==========================================
+ Hits         3062     3199     +137     
- Misses        438      440       +2     
- Partials       46       48       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.