Skip to content

Add missing deque methods #2812

Closed
Closed

Activity

  1. OhBonsai commented on Aug 9, 2021

    @OhBonsai
    Contributor

    Hi, I would like to do some work on this. Could you @DimitrisJim get me up to speed quickly on this? :)

  2. DimitrisJim commented on Aug 9, 2021

    @DimitrisJim
    MemberAuthor

    Of course @OhBonsai. You can start by taking a look at the other methods implemented for deque, these will give you an idea on how to manipulate the underlying VecDeque to get the behavior needed. You can drop in at gitter and ask about anything if you get stuck at some point.

  3. DimitrisJim commented on Aug 10, 2021

    @DimitrisJim
    MemberAuthor

    Hey @OhBonsai , pick one of the issues linked and I can assign it to you and guide you through anything you need over there.

  4. whgusdn321 commented on Aug 11, 2021

    @whgusdn321
    Contributor

    @OhBonsai Hello I'm interested in this issue also. If you don't mind, I would like to contribute some of them. would it be possible? and if it's okay, what are you going to implement among these? I'll implement the ones you don't pick.
    ( collections.deque doesn't have add method #2835
    collections.deque doesn't have a bool method. #2837
    collections.deque doesn't have iadd method. #2838
    collections.deque doesn't have imul method. #2839
    collections.deque does not have rmul method #2834
    collections.deque doesn't have reversed method #2836)

  5. OhBonsai commented on Aug 11, 2021

    @OhBonsai
    Contributor

    @whgusdn321 Wows, It's seem like i can ride your coattails 😄 .

    I think i could choosebool. It looks simpler~. BTW add/iadd/imul/rmul have been assigned~. Just reversed leaved 😢

  6. youknowone commented on Aug 11, 2021

    @youknowone
    Member

    general tip: list and tuple have similar implementations for similar magic methods.

  7. hk1997 commented on Sep 4, 2021

    @hk1997

    Hi,
    Is it still open?

  8. DimitrisJim commented on Sep 5, 2021

    @DimitrisJim
    MemberAuthor

    Ah, I'm afraid not, the last part of this issue is being worked on by someone atm.

    I'll try and see if I can find any other easy issues soon though, so, be on the look-out for them.

  9. DimitrisJim commented on Sep 11, 2021

    @DimitrisJim
    MemberAuthor

    @hk1997 see #3039 for a couple more good first issues you could try and take-up.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions