Skip to content
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

Ant Colony Optimization #25

Open
wants to merge 28 commits into
base: master
Choose a base branch
from
Open

Conversation

natclaret
Copy link

No description provided.

@MichielStock
Copy link
Owner

  • simplify the name of the notebook
  • please add some explanation of the algorithm
  • you might use Graphs.jl for visualization and to work with graphs in general, it might simplify your code a lot!
  • AntColonyOptimization => antcolonyoptimizaton not capitals in function or variable names! 😠
  • round(rand()*10,2) => floor(Int, ...)?
  • in updatePheromones: simplify by pheromones .= clamp.(pheromones, 1e-5, 1e5)
  • you might check 'Algorithms for Optimization' for a simpler example of this algorithm

In general: tidy up your notebook and make it a bit simpler. Explain your algorithm well using text and clean code!

Keep up the good work!

  • M

@natclaret natclaret changed the title Ant Colony Optimization Ant Colony Optimization Jan 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants