-
Notifications
You must be signed in to change notification settings - Fork 666
Closed
Labels
enhancementNew feature or requestNew feature or requestreleasedcode merged into repo AND released to Pypicode merged into repo AND released to Pypi
Description
I added my 10th panel today and hit two places where the "validators" rejected the plot.
Easy fix: just increase (or remove) the upper limit check. I increased them to 20 in my local source and everything still works.
Any reason the limit is there in the first place?
mplfinance/src/mplfinance/_arg_validators.py
Line 304 in 385b27c
return panid in ['main','lower'] or (isinstance(panid,int) and panid >= 0 and panid < 10) |
mplfinance/src/mplfinance/plotting.py
Lines 207 to 209 in 1cc114a
'panel_ratios' : { 'Default' : None, | |
'Validator' : lambda value: isinstance(value,(tuple,list)) and len(value) <= 10 and | |
all([isinstance(v,(int,float)) for v in value]) }, |
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or requestreleasedcode merged into repo AND released to Pypicode merged into repo AND released to Pypi