Renaming for descriptiveness

euro-hpc-pl / SpinGlassEngine.jl

Apache License 2.0

1 stars 1 forks source link

Renaming for descriptiveness #29

Closed annamariadziubyna closed 1 week ago

annamariadziubyna commented 1 month ago

In this new version, we have made several updates to improve the codebase:

Removed MpsAnnealing: The MpsAnnealing structure has been completely removed.
beta is now a scalar: The beta parameter has been changed from vector form to a scalar.
Renaming: We've made additional changes to names throughout the code to ensure they are more descriptive and intuitive.
Args to kwargs conversion: Continued conversion of function arguments (args) to keyword arguments (kwargs) for better clarity and usage.

coveralls commented 1 month ago

Pull Request Test Coverage Report for Build 10236145432

Details

54 of 83 (65.06%) changed or added relevant lines in 8 files are covered.
5 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.06%) to 75.187%

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
src/util.jl	0	1	0.0%
src/search.jl	11	14	78.57%
src/contractor.jl	23	48	47.92%
<!--	Total:	54	83	65.06%	-->

Files with Coverage Reduction	New Missed Lines	%
src/contractor.jl	5	66.8%
<!--	Total:	5		-->

Totals
Change from base Build 10213522947:	0.06%
Covered Lines:	1306
Relevant Lines:	1737

💛 - Coveralls

annamariadziubyna commented 2 weeks ago

Hi, I wanted to kindly ask if there has been any progress on reviewing the pull request I submitted. Once the pull request is merged into the master branch, you can proceed with resolving the two issues I raised to ensure they're aligned with the latest changes. Thank you!