iamaya2 / MatHH

A Matlab-based Hyper-Heuristic framework
MIT License
6 stars 1 forks source link

Clean RBSHH class #59

Open iamaya2 opened 11 months ago

iamaya2 commented 11 months ago

Check the whole class and remove duplicates and useless elements... For example, step and solveInstance have duplicate actions... check which one to remove or to move to another kind of method (private?). The idea is to reduce the margin for confusion..

Also, there are some variables that seem out-of-place. For example, the heuristicVector2 variable within the solveInstance method.. check with @alonsovelam if those variables belong to him.

Pending stuff:

iamaya2 commented 11 months ago

For example, the heuristicVector2 variable within the solveInstance method.. check with @alonsovelam if those variables belong to him.

Validated with @alonsovelam and this variable can be removed...

iamaya2 commented 10 months ago
  • [ ] Remove properties that have been commented out (they are unused)

Most useless properties have been removed from the class. The hasInitialized property was upgraded to parent class.

Pending: check if the heuristicVector property can be removed after trimming the classes (it is currently used by some methods, but it may be omitted).

iamaya2 commented 10 months ago
  • [ ] getInstances

Removed!

iamaya2 commented 10 months ago
  • [ ] step
  • [ ] _solveInstanceSetnoCloning

Removed!