Skip to content

Delete double_ended_queue.py #1208

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 4 commits into from
Closed

Conversation

cclauss
Copy link
Member

@cclauss cclauss commented Sep 30, 2019

Fixes #1200

As discussed on #1200 this file does not contain an algorithm as it does not implement a double ended queue but instead just uses the standard library's DEQ. @paulo9428 @cozek Your reviews please.

Fixes #1200 

As discussed on #1200 this file does not contain an algorithm as it does not implement a double ended queue but instead just uses the standard library's DEQ.  @paulo9428 @cozek, @AnupKumarPanwar Your reviews please.
@ghost
Copy link

ghost commented Sep 30, 2019

I totally agree with that.

@cozek
Copy link
Member

cozek commented Sep 30, 2019

I totally agree with that.

Hey, @paulo9428. I think that it's great that you want to commit to open source. And I want to see your pull request be merged! Please do not be discouraged.

I would advise you to read up on how to implement queues first. Once you are comfortable with that and have a clear understanding of how it's working, try to implement deque on your own. After that, you should write comments that clearly describe the logic behind your code in the format described in the contribution guide of this repo (along with doctests, but that's for later). You could, for instance, take reference from the other queues already contributed to this repo and build on that (with due credits to the authors). Maybe even pick up jupyter notebook and write a story on deques. The main point of your contribution is that it need not be industry level perfect, but it should educate the person reading your code about deques and how it works.

I hope we see another PR from you soon! Keep coding!

@ghost
Copy link

ghost commented Oct 1, 2019

thank you! you are very kind person. i am encouraged by your sincere advice

@cclauss
Copy link
Member Author

cclauss commented Oct 1, 2019

@paulo9428 We look forward to your next contribution... Thanks for joining our team!

@github-actions
Copy link

github-actions bot commented Dec 3, 2019

Stale pull request message

@cclauss cclauss closed this Dec 6, 2019
@cclauss cclauss deleted the Remove-double_ended_queue.py branch December 6, 2019 06:39
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