Skip to content
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

Add Priority Queue #131

Closed
czgdp1807 opened this issue Mar 9, 2020 · 11 comments · Fixed by #205
Closed

Add Priority Queue #131

czgdp1807 opened this issue Mar 9, 2020 · 11 comments · Fixed by #205

Comments

@czgdp1807
Copy link
Member

Description of the problem

Example of the problem

References/Other comments

.. [1] https://en.wikipedia.org/wiki/Priority_queue

@sarthakforwet
Copy link

i wanna take this issue..

@czgdp1807
Copy link
Member Author

@sarthakforwet Sure you can. The APIs are defined on the Wikipedia page, so a PR can be made directly.

@sarthakforwet
Copy link

@czgdp1807 should I have to add a new branch for the pull request to be made because from the current developer branch my previous PR is showing when I click on New PR

@czgdp1807
Copy link
Member Author

@Disha5harma
Copy link

@czgdp1807 Is this issue still available?

@czgdp1807
Copy link
Member Author

@Disha5harma If you can make a better PR than #132 then feel free to do so. The best one will be kept opened and rest will be closed in its favour. Add PriorityQueue to queue.py.

@Disha5harma
Copy link

Disha5harma commented Mar 12, 2020 via email

@abhishekgupta368
Copy link

I want to work on this issue, can you please assign me?

@czgdp1807
Copy link
Member Author

Hi @abhishekgupta368
Feel free to start work on this issue. Please see our issue policy to get to know how to work on issues.
As you will be adding a new data structure, please go through https://github.com/codezonediitj/pydatastructs/wiki/Plan-of-Action-for-the-Projects for the steps.

@aravind-karthikeyan
Copy link
Contributor

Is this still open?

@czgdp1807
Copy link
Member Author

@aravind-karthikeyan No one has picked up this issue. Please start by going through the discussion held here in various PRs opened for this issue.

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 a pull request may close this issue.

6 participants