Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Update learning.py#417

Closed
BesanHalwa wants to merge 7 commits intoaimacode:masteraimacode/aima-python:masterfrom
BesanHalwa:masterBesanHalwa/aima-python:masterCopy head branch name to clipboard
Closed

Update learning.py#417
BesanHalwa wants to merge 7 commits intoaimacode:masteraimacode/aima-python:masterfrom
BesanHalwa:masterBesanHalwa/aima-python:masterCopy head branch name to clipboard

Conversation

@BesanHalwa
Copy link
Contributor

converted method sample(self) to propety and removed the call in the statement return self.sampler

converted method sample(self) to propety and removed the call in the statement return self.sampler
minor code formatting.
Implemented Genetic Algoritm in search.py and search.ipynb
@Chipe1
Copy link
Contributor

Chipe1 commented Apr 3, 2017

search.ipynb has an unnecessarily long output which, along with images, makes it 2.45 MB!
It is better to have the images in the repo itself rather than fetching them from external sources.
You might have to change the newline character on the editor you used. It replaced all '\n' with '\r\n'. Notepad on windows does that. You can use notepad++ or any other non-windows editor/IDE like emacs instead.

…ording to flake8. Demonstrated the working of GA iin search.ipynb.
…ording to flake8. Demonstrated the working of GA iin search.ipynb.
Removed the image links and included the images used in the image
folder. Reduced the file size from 2.4 MB to 183 KB. For every 21 print
statements that we previously had, now we have 2, only printing the
fittest individual in each generation. The instructions has been made to
get the detailed output.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments

Close sidebar
Morty Proxy This is a proxified and sanitized view of the page, visit original site.