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

Skip to content

Implemented GA in notebook search.ipynb #488

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 13 commits into from

Conversation

BesanHalwa
Copy link
Contributor

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.

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
…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.
@Chipe1
Copy link
Contributor

Chipe1 commented Apr 9, 2017

@Agent-Pandit You might have to add .idea in your gitignore or not add it to your commit. There are a few xml files which I think are created by your IDE. It also has new line as '\r\n' instead of '\n' as mentioned in #417

@BesanHalwa
Copy link
Contributor Author

@Chipe1 thanks for pointing it out. Do I need to make another PR or this will work?

@Chipe1
Copy link
Contributor

Chipe1 commented Apr 9, 2017

I'm not sure how to uncommit changes. Maybe you can delete the files in a new commit and also fix the merge conflict.

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