Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Update learning.py #417

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

Closed
wants to merge 7 commits into from
Closed

Update learning.py #417

wants to merge 7 commits into from

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