Skip to content

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