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

Feature Request: vlines argument None or False or ... #239

Open
twopirllc opened this issue Aug 15, 2020 · 3 comments
Open

Feature Request: vlines argument None or False or ... #239

twopirllc opened this issue Aug 15, 2020 · 3 comments
Labels
enhancement New feature or request

Comments

@twopirllc
Copy link

Hello @DanielGoldfarb,

Mplfinance is looking great and I have adopted it in my Example Notebook for my Pandas TA Library.

Is your feature request related to a problem? Please describe.
Simple value to pass into vlines argument so it passes without error.

Describe the solution you'd like
Some value like None or False or whatever you deem appropriate for the package requirements.

Describe alternatives you've considered
I created an invisible out of chart range data to bypass the error. It's a hack but seems unnecessary.

if amat_sr:
    vlines_ = dict(vlines=amat_sr, alpha=0.1, colors="
else:
    # Hidden because vlines needs valid arguments even if None 
    vlines_ = dict(vlines=mpfdf.index[0], alpha=0, colors="white")

Additional context
I made a semi-dynamic Panels Chart Class Example Notebook for my Pandas TA Library. The plot method is located at the bottom of Input Cell 11. You can play around with the Chart class arguments in Input Cell 12.

Thanks for your time,
Kevin

@twopirllc twopirllc added the enhancement New feature or request label Aug 15, 2020
@DanielGoldfarb
Copy link
Collaborator

Hi Kevin,
Sounds reasonable.

So then, just to be clear, if we do this your code will become:

if amat_sr:
    vlines_ = dict(vlines=amat_sr, alpha=0.1, colors="...
else:
    vlines_ = None

Is that correct? (i.e. that's what you expect/want?)
--Daniel

@twopirllc
Copy link
Author

Hello Daniel,

That is correct. That would be helpful. 😎

Thanks,
Kevin

@DanielGoldfarb
Copy link
Collaborator

oops. clicked wrong button.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants