push5280
201 posts
msg #150507
- Ignore push5280 |
1/26/2020 7:42:49 AM
I'm having an issue with syntax in this scan and hoped that one of the coding champions could help. I think that I have narrowed it to one of two lines of code. The problem lies either in this set phrase under /*my indicators*/ set{mymac%,close / ma(126)} or this line under /*criteria*/ mymac% crossed .8 from below.
The full scan is below, any assistance you can provide is appreciated
|
mahkoh
1,065 posts
msg #150508
- Ignore mahkoh
modified |
1/26/2020 8:31:35 AM
Use "crossed above 0.8" in the second snippet
|
JoeyVinyl
125 posts
msg #150511
- Ignore JoeyVinyl |
1/26/2020 10:57:12 AM
I clicked on it (thanks for making it clickable), made the change that mahkoh suggested, and the first syntax error was this line: set{myslp, count( 7 day slope of PVI > -.01,2)}
I'm far from a coding expert here but can you count how many days a slope of anything was above a certain number? I think I'll play with that and see what I come up with.
|
JoeyVinyl
125 posts
msg #150512
- Ignore JoeyVinyl
modified |
1/26/2020 11:18:25 AM
Okay, got it. You have a - sign in that line in front of the .01. I played around with PVI and the lowest number I got was 0. When I deleted that sign in your code the syntax error went away. It didn't show any other syntax errors but no stocks made it through the filter.
set{myslp, count( 7 day slope of PVI > -.01,2)}
Change -.01 to .01
|
xarlor
587 posts
msg #150513
- Ignore xarlor |
1/26/2020 11:47:57 AM
Without changing anything, results are returned for 01/24/2020 and further back. Do these results not match what you were expecting?
|
JoeyVinyl
125 posts
msg #150521
- Ignore JoeyVinyl |
1/26/2020 8:40:54 PM
@xarlor
That's odd. I just tried it again without changing anything and didn't get results for Thursday or Friday, (1/23 or 1/24) but did get them for Wednesday, 1/22, and before without any syntax errors. I am definitely NOT one of the coding champions here so if push5280 is still having problems with it then I don't know why but I hope they get them resolved.
@push5280, SF support has found errors in my code when I email it to them that I didn't see, so if you're still having problems that's what I'd recommend.
|
xarlor
587 posts
msg #150523
- Ignore xarlor |
1/26/2020 8:54:34 PM
Apologies, I meant to say 01/22/2020 and further back. I do not get results for 1/23/2020 or 1/24/2020 either.
|
KSK8
561 posts
msg #150536
- Ignore KSK8 |
1/27/2020 11:05:11 AM
Do you mind explaining the 'max shares' component as well as the purpose of the filter's use of the 'PVI' ? Genuinely curious.
|
byebyemoney
31 posts
msg #150650
- Ignore byebyemoney |
2/3/2020 3:39:45 PM
no syntax errors for me, worked fine
|