search

MyIntelligenceAgency / Lean

Lean Algorithmic Trading Engine by QuantConnect (C#, Python, F#)
https://lean.io
Apache License 2.0
0 stars 32 forks source link

Paramétrage fichier config #14

Closed fbamigbola closed 1 year ago

fbamigbola commented 1 year ago

Configuration du fichier config.json en remplacant le nom de l'algo par algowarriors et duplication de BasicTemplateAlgorithm en algowarriors

jsboige commented 1 year ago

Bonjour, le fichier de config à personnaliser est celui situé dans le répertoire bin/debug, qui n'est pas archivé sur le dépôt. Sur les 3 fichiers modifiés de cette PR, seul le fichier AlgoWarriors est légitime. Pouvez-vous rétablir les 2 autres ? D'autre part, il faudrait également renommer la classe BasicTemplateAlgorithm pour lui donner le nom du fichier. Enfin, l'interface IRegressionAlgorithmDefinition et tout ce qu'il y a entre la ligne 65 et 114 ne sert qu'aux tests unitaires et peut être supprimé. Merci d'avance pour ces ajustements.

fbamigbola commented 1 year ago

Bonjour Monsieur, On a fait un pull request de nouveau. On voudrait savoir si vous l'avez vu s'il vous plaît. Cordialement

From: Jean-Sylvain Boige @.> Sent: November 16, 2023 13:53 To: MyIntelligenceAgency/Lean @.> Cc: fbamigbola @.>; Author @.> Subject: Re: [MyIntelligenceAgency/Lean] Paramétrage fichier config (PR #14)

Bonjour, le fichier de config à personnaliser est celui situé dans le répertoire bin/debug, qui n'est pas archivé sur le dépôt. Sur les 3 fichiers modifiés de cette PR, seul le fichier AlgoWarriors est légitime. Pouvez-vous rétablir les 2 autres ? D'autre part, il faudrait également renommer la classe BasicTemplateAlgorithm pour lui donner le nom du fichier. Enfin, l'interface IRegressionAlgorithmDefinition et tout ce qu'il y a entre la ligne 65 et 114 ne sert qu'aux tests unitaires et peut être supprimé. Merci d'avance pour ces ajustements.

— Reply to this email directly, view it on GitHubhttps://github.com/MyIntelligenceAgency/Lean/pull/14#issuecomment-1814382726, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AUCJSI73CA4KUMYZOQAD663YEYEE3AVCNFSM6AAAAAA7OC6OMCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMJUGM4DENZSGY. You are receiving this because you authored the thread.Message ID: @.***>

jsboige commented 1 year ago

Bonjour, j'ai bien pris note de votre nouvelle PR, mais je vous y ai mis un commentaire: 2 fichiers y sont modifiés, celui en ajout de votre algorithme, et, probablement par mégarde, celui du fichier de config à la racine du projet launcher. Est-ce que vous pouvez corriger cette modification là qui n'a pas lieu d'être avant que je fusionne votre PR? Une fois les validations remontées sur le serveur, il n'est pas bien possible de les annuler, mais vous pouvez toujours faire une nouvelle validation de rectification.

fbamigbola commented 1 year ago

Bonjour Monsieur, Merci pour votre retour. On a supprimé la sur laquelle était situé la modification des deux fichiers. Cette fois si c'est juste 1 fichier qui a été modifié celui de l'algowarriors seul. Cordialement.

From: Jean-Sylvain Boige @.> Sent: November 21, 2023 13:05 To: MyIntelligenceAgency/Lean @.> Cc: fbamigbola @.>; State change @.> Subject: Re: [MyIntelligenceAgency/Lean] Paramétrage fichier config (PR #14)

Bonjour, j'ai bien pris note de votre nouvelle PR, mais je vous y ai mis un commentaire: 2 fichiers y sont modifiés, celui en ajout de votre algorithme, et, probablement par mégarde, celui du fichier de config à la racine du projet launcher. Est-ce que vous pouvez corriger cette modification là qui n'a pas lieu d'être avant que je fusionne votre PR? Une fois les validations remontées sur le serveur, il n'est pas bien possible de les annuler, mais vous pouvez toujours faire une nouvelle validation de rectification.

— Reply to this email directly, view it on GitHubhttps://github.com/MyIntelligenceAgency/Lean/pull/14#issuecomment-1820793428, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AUCJSIYGIUBVTNO75BIF26LYFSKI3AVCNFSM6AAAAAA7OC6OMCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMRQG44TGNBSHA. You are receiving this because you modified the open/close state.

B

jsboige commented 1 year ago

Je vois toujours 2 fichiers en modification de mon côté: https://github.com/MyIntelligenceAgency/Lean/pull/13/files C'est pas pour vous embêter d'autant que la modification est bénine, mais j'aime autant qu'on ne touche pas trop les fichiers par défaut de Lean à moins que ça soit pour des corrections. Le cas échéant, ça sera plus simple pour leur faire des pull requests.

fbamigbola commented 1 year ago

Bonjour Monsieur, On a des données sur BAC, AAPL, et IBM du 01/01/2001 au 31/12/2020 sur Quantconnect gratuites avec un timeframe heure....Dans les données que vous avez fournies, on a pareil. Du coup si on doit travailler sur un portefeuille actions ca ne pose plus de soucis de données. Cordialement.

From: Jean-Sylvain Boige @.> Sent: November 16, 2023 13:53 To: MyIntelligenceAgency/Lean @.> Cc: fbamigbola @.>; Author @.> Subject: Re: [MyIntelligenceAgency/Lean] Paramétrage fichier config (PR #14)

Bonjour, le fichier de config à personnaliser est celui situé dans le répertoire bin/debug, qui n'est pas archivé sur le dépôt. Sur les 3 fichiers modifiés de cette PR, seul le fichier AlgoWarriors est légitime. Pouvez-vous rétablir les 2 autres ? D'autre part, il faudrait également renommer la classe BasicTemplateAlgorithm pour lui donner le nom du fichier. Enfin, l'interface IRegressionAlgorithmDefinition et tout ce qu'il y a entre la ligne 65 et 114 ne sert qu'aux tests unitaires et peut être supprimé. Merci d'avance pour ces ajustements.

— Reply to this email directly, view it on GitHubhttps://github.com/MyIntelligenceAgency/Lean/pull/14#issuecomment-1814382726, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AUCJSI73CA4KUMYZOQAD663YEYEE3AVCNFSM6AAAAAA7OC6OMCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMJUGM4DENZSGY. You are receiving this because you authored the thread.Message ID: @.***>

jsboige commented 1 year ago

Super si vous avez les données qu'il vous faut. Pas de souci naturellement pour que vous partiez sur un portefeuille d'action ou encore mieux un portefeuille hybride vous permettant de tester des stratégies pondérant des actifs différents. Juste assurez-vous bien que vos simulations tournent en local. Et pour l'autre pull request, il y a toujours cette histoire de fichier indésirable, mais bon continuez à pousser des validations et nous fusionnerons en temps voulu.