-
Notifications
You must be signed in to change notification settings - Fork 1
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
Make the names more understandable #22
Comments
@marnoldus @greg-neustroev as mentioned here (#13 (comment)) here are some extras for the existing code. Most of them are just naming suggestions, but maybe the question (for l31 in |
@gnawin what I intend to store in the results are the solutions themselves and their objective value to the original problem. The optimality gap constraint (original objective) will be something like: |
Here are some (minor) suggestions that will improve the readability of the code while going through the repo.
https://github.com/TulipaEnergy/NearOptimalAlternatives.jl/blob/main/src/alternative-optimisation.jl:
so the sum of the distance between the new solution and each of the previous solutions is maximized
to the commenthttps://github.com/TulipaEnergy/NearOptimalAlternatives.jl/blob/main/src/results.jl:
push!(results.objective_values, objective_value(model))
? What do we want to store here?https://github.com/TulipaEnergy/NearOptimalAlternatives.jl/blob/2-mga-metaheuristics/src/generate-alternatives.jl:
The text was updated successfully, but these errors were encountered: