Skip to content

Python 3.x support#75

Open
romanbsd wants to merge 4 commits into
QuantSoftware:masterfrom
romanbsd:master
Open

Python 3.x support#75
romanbsd wants to merge 4 commits into
QuantSoftware:masterfrom
romanbsd:master

Conversation

@romanbsd

Copy link
Copy Markdown

I updated the QSTK for Python 3.x support. I'm using it for the coursera course. So far I hadn't had any problems.

@romanbsd

Copy link
Copy Markdown
Author

Is it going to be merged?

@pmiddend

pmiddend commented Jun 2, 2018

Copy link
Copy Markdown

Note that the tests don\t seem to run:

 tradesim() missing 3 required positional arguments: 'alloc', 'df_historic', and 'f_start_cash'

Though I'm not sure this is your fault or if the original repo's tests don't run, either.

@romanbsd

romanbsd commented Jun 6, 2018

Copy link
Copy Markdown
Author

@pmiddend I'll check. How do you run the tests?

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.

3 participants