You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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:
PascalCasing for all public identifiers except parameters (as described here)
Interface names should begin with I
Comments for documentation and Intellisense (as described here)
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
The text was updated successfully, but these errors were encountered:
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.
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.
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
The text was updated successfully, but these errors were encountered: