Skip to content

Added utils.ipynb - #765

Closed
SkAdilina wants to merge 1 commit into
aimacode:masterfrom
SkAdilina:master
Closed

Added utils.ipynb#765
SkAdilina wants to merge 1 commit into
aimacode:masterfrom
SkAdilina:master

Conversation

@SkAdilina

Copy link
Copy Markdown
Contributor

utils.ipynb now contains Queue, FIFOQueue and PriorityQueue

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

apb7 commented Feb 24, 2018

Copy link
Copy Markdown
Contributor

@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
Copy Markdown
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

apb7 commented Feb 24, 2018

Copy link
Copy Markdown
Contributor

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

@antmarakis

Copy link
Copy Markdown
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

norvig commented Feb 24, 2018

Copy link
Copy Markdown
Member

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

@antmarakis

Copy link
Copy Markdown
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