Give screeplot an ax argument

[loftusa]

function here.

Currently no way to specify an ax, most other plotting functions have that unless there's a good reason not to. I don't see a good reason for this one not to have it. Should be pretty easy to add on.

[himanshu007-creator]

Hi, i would like to work on this issue, shall i take it?!🙂

[loftusa]

Hi, i would like to work on this issue, shall i take it?!🙂

Sure. Just want to be able to specify a matplotlib axis if i'm using the screeplot function.

[rookiexiong7]

Is this work still ongoing?I am a beginner and would like to give it a try.

[kidkoder432]

I think you can still PR, as it hasn't been closed. I'm going to try it as well :)

[MuLIAICHI]

Greeting, did anyone try to PR that issue ?

[kidkoder432]

My PR is awaiting review.

…

On Sun, Sep 17, 2023, 8:22 AM M.liaichi ***@***.***> wrote: Greeting, did anyone try to PR that issue ? — Reply to this email directly, view it on GitHub <#840 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AOYIALR5AB7JDVKYANSGY2TX24ISRANCNFSM5EK3IK6Q> . You are receiving this because you commented.Message ID: ***@***.***>

[MuLIAICHI]

They should really bay attention here !

[kidkoder432]

I'm sorry, who are you referring to?

…

On Mon, Sep 25, 2023, 4:24 PM M.liaichi ***@***.***> wrote: They should really bay attention here ! — Reply to this email directly, view it on GitHub <#840 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AOYIALTGN4YA7Y3WDOG2II3X4IHCRANCNFSM5EK3IK6Q> . You are receiving this because you commented.Message ID: ***@***.***>

[bdpedigo]

FYI this was closed automatically since the PR was merged #1048 :)

[MuLIAICHI]

Ok :)