Closed edwinb-ai closed 3 years ago
Merging #49 (45fee96) into master (dbe115a) will not change coverage. The diff coverage is
100.00%
.
@@ Coverage Diff @@
## master #49 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 9 9
Lines 220 255 +35
=========================================
+ Hits 220 255 +35
Impacted Files | Coverage Δ | |
---|---|---|
src/solvers/pso.jl | 100.00% <100.00%> (ø) |
|
src/solvers/simulated_annealing.jl | 100.00% <100.00%> (ø) |
|
src/utils.jl | 100.00% <100.00%> (ø) |
|
src/optimize.jl | 100.00% <0.00%> (ø) |
|
src/types/solvers.jl | 100.00% <0.00%> (ø) |
|
src/types/population.jl | 100.00% <0.00%> (ø) |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update dbe115a...45fee96. Read the comment docs.
There seemed to be a problem with the search bounds for Simulated Annealing, where the bounds were not taken into account at all.
Also, the PSO implementation received some minor tweaks. Now the velocities are always clamped between the range [-v_max, v_max].