-
Notifications
You must be signed in to change notification settings - Fork 3.9k
added exmples to notebooks LOGIC & PLANNING. #424
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
Conversation
in pseudo code the sequence of arguments is " WEIGHTED-SAMPLE-WITH-REPLACEMENT(N, S, W)"
in pseudo code the sequence of arguments is " WEIGHTED-SAMPLE-WITH-REPLACEMENT(N, S, W)" same must follow in function "particle_filtering" in the file probability.py
weight_sample_with_replacement sequence of args
weighted_sample_with_replacement sequence of args
added double_tennis_problem from chapter 11 , figure 11.10
added missing space after comma
added missing space after comma
added missing space after comma
added missing space after comma
pep8 changes, showed flake8 errors
added missing whitespace after comma
added space after comma
added space after comma
added space after comma
added space after comma
added double_tennis_problem
In the pseudocode figure 21.8, the first 'if' starts with argument 's', which is the previous state, not s1(i.e, the current state).
Update rl.py
# Conflicts: # planning.py
# Conflicts: # planning.py
added blocks_world.png
updated fol_fc_ask()
Nice work, but there are quite a lot of issues here, I'm afraid.
These are the technical issues I noticed in your PR. If you fix them, this will be fine. I want to mention another thing too though. Here you are not explaining what is happening. Sure, you are showing how to use the code, but you are not telling the reader what is happening. I will give just one example, but that is evident in pretty much everywhere. The following code defines the initial state for the problem def air_cargo(): init = [expr('At(C1, SFO)'), expr('At(C2, JFK)'), expr('At(P1, SFO)'), ... ... ... This shows how to define the air cargo problem, but it does not help the reader. You should write why does it define the problem. Explain why is The only way for the reader to understand what this means is to go read the book instead and try figure things on their own, which kind of defeats the purpose of explanatory examples. While this project is complementary to the book, the examples should have some substance of their own. Otherwise people would just take a look at the code and the tests to get a feel of how to get this to work. In the above case, you are only showing how to define one single problem. What your example should do is show the reader how to define any problem. Of course, this is up to @norvig, but personally I feel this should not be merged as is. |
While I do agree with with @MrDupin , I think this PR can be merged as it serves as a starting point which can be improved upon. The only major problems which I see in this PR are that some of the definitions(Air Cargo, Spare Tire) have only code but no explanation, and that it has too many unrelated commits and changes. |
Just to clarify, I'm not saying the changes shouldn't be merged. They are helpful and they can definitely be merged, but not before the technical issues are dealt with (at least in my opinion). I agree with @Chipe1 about splitting the PR. I would suggest you fix the issues first (maybe even posting them here for review) and then make the new PRs. That way the new PRs will not get bloated with patches. |
# Conflicts: # planning.py
# Conflicts: # planning.py
@sofmonk: This is better. I only had time to skim over it, but I noticed some small issues I will quickly point out:
I would like to see though short introduction to the problems. For the air-cargo problem, drop a couple of lines on explaining the problem at hand. |
# Conflicts: # planning.py
# Conflicts: # planning.py
@sofmonk One way to fix it is to diff your current branch with master, create a fresh branch which is up to date with master and apply the diff to the newly created branch. |
Sir, I found some examples/diagrams from the other websites good, but I didn't include them as I was skeptical if we can add content from other websites. Please let me know so that I can add them?