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.

Sign up for GitHub

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

Jump to bottom

Add Environment.java to AIMA4e #370

Closed
wants to merge 6 commits into from
Closed

Add Environment.java to AIMA4e #370

wants to merge 6 commits into from

Conversation

ritwik12
Copy link
Contributor

This PR consists :

  1. Add Environment in README.md.
  2. Add Environment.java.

@ritwik12
Add Environment to AIMA4e
0229025 
Add Environment.java to AIMA4e

Add Environment.java to AIMA4e
@ritwik12
Copy link
Contributor Author

solves issue #371

@okulev
Copy link

okulev commented Mar 17, 2018

The JavaDoc comments in 0229025 are incorrectly placed.

@ritwik12
Copy link
Contributor Author

Sorry, @okulev I am not able to understand. Can you please point out the improvement?

@okulev
Copy link

okulev commented Mar 17, 2018

The JavaDoc comment of addAgent method should be JavaDoc for getAgents. JavaDoc of removeAgent should be moved to addAgent, and so on.

@ritwik12
Copy link
Contributor Author

Thanks, @okulev for the correction. I corrected it

@ritwik12 ritwik12 closed this Mar 17, 2018
@ritwik12 ritwik12 deleted the patch-4 branch March 17, 2018 20:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@ritwik12 @okulev @utsavoza