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

Fixed tensorflow upgrade issues #142

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

mridulrb
Copy link

@mridulrb mridulrb commented Oct 1, 2021

No description provided.

@0xdarkman
Copy link

0xdarkman commented Oct 8, 2021

well done in regards to code changes itself.
how about the model performance? have you noticed any changes?

@0xdarkman
Copy link

you are missing xarray in requirements.txt

@mridulrb
Copy link
Author

mridulrb commented Oct 9, 2021

you are missing xarray in requirements.txt

@0xdarkman I have updated the requirements.txt to install xarray

@mridulrb
Copy link
Author

mridulrb commented Oct 9, 2021

well done in regards to code changes itself. how about the model performance? have you noticed any changes?

@0xdarkman Thanks for the feedback. The results I got are as follows -

algorithm portfolio value sharpe ratio max drawdown positive periods negative periods postive day negative day postive week negative week average
nntrader 46.732177 0.081992 0.224624 1223 1553 2121 655 2752 24 1.001568
olmar 1.704328 0.019966 0.802504 1295 1475 1209 1567 1264 1512 1.000491
ons 1.617763 0.029253 0.220506 1395 1381 1596 1180 1965 811 1.000196

Figure_1

@fridary
Copy link

fridary commented Nov 7, 2021

@mridulrb bro, on what dates did you get such large portfolio value? I backtested PGPortfolio on the default config and got 1.016706 BTC during 2019/07/01 - 2021/07/01 (test_portion=0.08). Nevertheless, I got 33.561700 BTC during 2015/07/01 - 2017/07/01

@mridulrb
Copy link
Author

@fridary The dates were 2015/07/01 - 2017/07/01.

@yidi666
Copy link

yidi666 commented Dec 1, 2021

hello @mridulrb , could u kindly provide ur updated codes with me via email [email protected] ? l have trouble in dealing with the bugs caused by the updating of pandas...(some attributions are deserted and l cannot fix it since l am a beginer of python) thank u very much !

@Nice-Zhang66
Copy link

你好@mridulrb您可以通过电子邮件[email protected]向我提供更新后的代码吗?我在处理pandas更新引起的bug时遇到了麻烦...(有些属性已废弃,我无法修复它,因为我是python的初学者)非常感谢你!

Hello, does your code work, I want to experiment with my own collected data, how should I use it, can you send me your code? Here is my email: [email protected] Thank you very much!
您好,你的代码能正常运行吗,我想用我自己采集的数据进行实验,该怎么用呢,能不能给我发一下您的代码呢?这是我的邮箱:[email protected],非常感谢您!

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.

5 participants