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

Skip to content

Added utils.ipynb #765

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 1 commit into from
Closed

Added utils.ipynb #765

wants to merge 1 commit into from

Conversation

SkAdilina
Copy link
Contributor

utils.ipynb now contains Queue, FIFOQueue and PriorityQueue

utils.ipynb now contains Queue, FIFOQueue and PriorityQueue
@apb7
Copy link
Contributor

apb7 commented Feb 24, 2018

@MrDupin and @SkAdilina : Do we actually require a notebook for utilities? We use notebooks to explain and visualize various ideas and concepts of AI whereas utilities help us to do so. They do not actually represent a concept. Please correct me if I am wrong!
Thanks!

@SkAdilina
Copy link
Contributor Author

@apb7 I am definitely not an expert on this, so I won't be able to correct you. However, this is where I saw that Queue is yet to be included in the Notebook, so I went ahead and added it.

@apb7
Copy link
Contributor

apb7 commented Feb 24, 2018

Okay, let's wait for @norvig and @MrDupin then :)

@antmarakis
Copy link
Collaborator

Hmm, you are right that it seems like we want a notebook for utils with the Queue in the README. I personally think such a notebook is not necessary, so maybe we want to remove Queue from the list of algorithms, or add a 'Not Needed' mark on the 'Notebook' section.

It is up to @norvig though.

@norvig
Copy link
Collaborator

norvig commented Feb 24, 2018

I don't think we need a notebook for utilities, including Queue. I think we can just import the code.

@antmarakis
Copy link
Collaborator

I am afraid since Mr. Norvig said we don't need a notebook for utilities, I have to close this.

Thanks for the work though. Keep it up!

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.

4 participants