aimacode / aima-csharp

C# implementation of algorithms from Russell And Norvig's "Artificial Intelligence - A Modern Approach"
MIT License
176 stars 86 forks source link

C# Conventions #6

Open hexagon06 opened 7 years ago

hexagon06 commented 7 years ago

While working through the book's Exercises and using the repository as a reference I noticed that the code does not use the standard conventions for C# I'm used to:

I'll be fixing those as I read through the book on my own fork to help me navigate through the code with more ease. Note that I will not change the implementation except for when something fails to build.

I created this issue here to see if you would be interested in the changes (through pull requests)

p.s. I saw some other issues with regard to compilation as well. I'll be fixing those too, but on a separate branch on the same fork

ShobhitLamba commented 7 years ago

Hello hexagon06.

Thank you for your interest in aima-csharp. Yes you can go ahead and make pull requests for the changes you wanna implement. I'll merge them once they check out.

Speedie007 commented 7 years ago

Good Day Hexagon06

I have already completed most of what you have discussed here and the points that you have raised including add and fixing the implementation of a ordered sorted Queue required for the search Algorithms which was missing component to implement the missing Informed Search algorithms. My intention was to complete the entire package and complete the remain two or three algorithms and then bring them forward for review to be added to the project. I also have an implementation of the Vacuum cleaner and the Eight Puzzle using the A*Star, Uniform-Cost-Search, Hill Climbing, and Simulated Annealing Algorithms Completed.

Speedie007 commented 7 years ago

If you would like to discuss the work i have completed so far i can be contacted Brendanw@mweb.co.za