Merge pull request #3 from MyIntelligenceAgency/master

[ELsamar]

Merge pull request #18 from ELsamar/master

Description

Related Issue

Motivation and Context

Requires Documentation Change

How Has This Been Tested?

Types of changes

• [ ] Bug fix (non-breaking change which fixes an issue)
• [ ] Refactor (non-breaking change which improves implementation)
• [ ] Performance (non-breaking change which improves performance. Please add associated performance test and results)
• [ ] New feature (non-breaking change which adds functionality)
• [ ] Breaking change (fix or feature that would cause existing functionality to change)
• [ ] Non-functional change (xml comments/documentation/etc)

Checklist:

• [ ] My code follows the code style of this project.
• [ ] I have read the CONTRIBUTING document.
• [ ] I have added tests to cover my changes.
• [ ] All new and existing tests passed.
• [ ] My branch follows the naming convention bug-<issue#>-<description> or feature-<issue#>-<description>

[jsboige]

Bonjour, merci pour votre algo dont j'ai pu prendre connaissance et que j'ai fusionné. J'ai rajouté un test "&& _priceWindow.IsReady" sur la condition de désengagement en cas de corrélation en dessous du seuil car j'ai remarqué que les résultats n'étaient pas bons, que l'algo vendait systématiquement dès qu'il achetait, et qu'en début de période c'était lié à ça. Ceci dit, j'ai pu voir que ça ne suffit pas, le comportement se poursuit même quand la rollingwindo est prête. Je ne suis pas allé suivant, il faudrait que vous repreniez votre logique en debug pour identifier le comportement deffectueux et trouver les bons paramètres. En tout cas ne prenez pas le risque que durant le même slice l'algo rentre et ressorte, ce qui a l'air de se passer maintenant.