Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Added utils.ipynb#765

Closed
SkAdilina wants to merge 1 commit intoaimacode:masteraimacode/aima-python:masterfrom
SkAdilina:masterCopy head branch name to clipboard
Closed

Added utils.ipynb#765
SkAdilina wants to merge 1 commit intoaimacode:masteraimacode/aima-python:masterfrom
SkAdilina:masterCopy head branch name to clipboard

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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.