Skip to content

Comments

Update RLTrader.py#152

Open
MichaelQuaMan wants to merge 1 commit intonotadamking:masterfrom
MichaelQuaMan:patch-1
Open

Update RLTrader.py#152
MichaelQuaMan wants to merge 1 commit intonotadamking:masterfrom
MichaelQuaMan:patch-1

Conversation

@MichaelQuaMan
Copy link

Issue
qs.report.html() function signature doesn't support file as an argument.

Call results in:

"TypeError: html() got an unexpected keyword argument 'file'". The correct argument is `output`. 

Support

qs.report.html() signature:

def html(returns, benchmark=None, rf=0., grayscale=False,
         title='Strategy Tearsheet', output=None, compounded=True,
         periods_per_year=252, download_filename='quantstats-tearsheet.html',
         figfmt='svg', template_path=None, match_dates=False):

and

Argument output usage:

    with open(output, 'w', encoding='utf-8') as f:
        f.write(tpl)

qs.report.html() signature doesn't support `file` as an argument; so, calling this results in: "TypeError: html() got an unexpected keyword argument 'file'". The correct argument is `output`.  See: https://github.com/ranaroussi/quantstats/blob/6aaa65c20bad4c364efa7623375901925c036b45/quantstats/reports.py#L57 and https://github.com/ranaroussi/quantstats/blob/6aaa65c20bad4c364efa7623375901925c036b45/quantstats/reports.py#L250
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 this pull request may close these issues.

1 participant