126; Correl/ Add empty periods seems to now work

If i disable this option in list should not be values with zero value. But they are there :) This bug is also in QA.

And when i check/uncheck this option there is no difference. 

Attachments
correl_bug.png
(331.47 KiB)
  • Votes 0
  • Project StrategyQuant X
  • Type Bug
  • Status Fixed
  • Priority Normal

History

IH
#1

clonex / Ivan Hudec

05.02.2020 17:05

Task created

h
#2

hankeys

05.02.2020 20:54
week 1, 7 and 15 are missing - in both strategies should be zero profit, because of that the rows are missing
b
#3

bentra

07.02.2020 05:55
Its normal. It only removes rows where both columns have a zero.
IH
#4

clonex / Ivan Hudec

07.02.2020 09:06
Im not sure if it true. There is no reason to calculate correl between yero occurrence. Ive implemented correl outside sqx but actually im computing correl only when both occurrences are happen. I'll try to modify intersect function in order to compute correl and ill will see
b
#5

bentra

07.02.2020 21:41
clonex: ah I think you are right, there shouldn't be a reason...
h
#6

hankeys

10.02.2020 07:15
there is a reason, imagine 2 strategies with this monthly profits


strat 1: 5,0,9

strat 2: 0,0,10


without empty periods it will calculate only correlation between months with trades (5,0) and (9,10)

and with empty periods it will calculate all months (5,0) (0,0) (9,10)


and thus the correlation must be different

h
#7

hankeys

10.02.2020 07:18
and second reason - you will have daily strategies with example only 200 trades in 17 years...so there are many months without trades and you want to compute the month correl


without empty periods the value of correl will be always very low and i am talking about monthly (strongest) correlation - weekly, daily - doesnt make sense


so for low number of trades you need to compute the correl with empty periods

IH
#8

clonex / Ivan Hudec

15.02.2020 22:39
but question remains: if i turn on o turn off this option there isnt difference
TT
#9

Tamas

21.02.2020 09:26

Status changed from New to In progress

TT
#10

Tamas

21.02.2020 09:35

Status changed from In progress to Fixed

It works as it is expected.


It is explained well here https://strategyquant.com/forum/topic/3772-help-file-for-quant-analyzer/


Votes: 0

Drop files to upload

or

choose files

Max size: 5MB

Not allowed: exe, msi, application, reg, php, js, htaccess, htpasswd, gitignore

...
Wait please