Skip to content
This repository was archived by the owner on Nov 13, 2023. It is now read-only.

Conversation

@Parth-DH
Copy link

Sample not included. Was not able to figure how to.

Sample not included. Was not able to figure how to.
@sambitdash
Copy link
Owner

@Parth-DH Thanks for your submission.

Without a test in AIMASamples, how can you be sure your code works? Can you review Vacuum sample and come up with a possible solution for a sample.

Copy link
Owner

@sambitdash sambitdash left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The algorithms do not match with what is there in the book. The intent is to keep the code closest to what is there in the book. Secondly, without a test sample it will be hard to find out if the code will work. So you will need to check-in possible sample code.


struct AndOrSearch{SQ,S<:State} <: SearchAlgorithm
explored::SET
AndOrSearch{SQ,S}() where {SQ, S <:State}= has_trait_queue(SQ{S}, NodeT{S, NT})
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use a space after comma

end

function AndSearch{S <:State}(problem, parent::NodeT{S, :greedy}, action)
state = result(problem, parent.state, action)
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does not match with the algorithm in the book 3e.

AndOrSearch{S<:State}(::S) = AndOrSearch{S}()

function OrSearch{S <:State}(problem, parent::NodeT{S, :greedy}, action)
state = result(problem, parent.state, action)
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does not match with the algorithm in the book 3e.

@Parth-DH
Copy link
Author

Really sorry about that @sambitdash . I had the second edition with me and could not find the algorithm so tried to implement by studying the functioning. I have arranged the 3rd edition and will correct it as soon as possible.

@sambitdash
Copy link
Owner

@Parth-DH please let me know if you are working on this or planning to work on it in near future. Since, there are some serious issues in the PR I will have to reject it otherwise.

@Parth-DH
Copy link
Author

Parth-DH commented Apr 8, 2018

@sambitdash I am not currently working on the PR as I have exams this week but will work on it as soon as the exams end.

@sambitdash
Copy link
Owner

Sure. I will leave it where you have left it. Best wishes to you for your exams.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants