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

Signal Export Generates Portfolio Targets for Non-Tradable Securities #8448

Closed
4 tasks done
AlexCatarino opened this issue Dec 5, 2024 · 2 comments · Fixed by #8453 or #8472
Closed
4 tasks done

Signal Export Generates Portfolio Targets for Non-Tradable Securities #8448

AlexCatarino opened this issue Dec 5, 2024 · 2 comments · Fixed by #8453 or #8472
Assignees
Labels

Comments

@AlexCatarino
Copy link
Member

Expected Behavior

Generate PortfolioTarget for a portfolio with Index.

Actual Behavior

Since Index is not supported (because it's not tradable), we get the following message:

Index security type is not supported by Collective2. Allowed security types: [Equity,Forex,Option,Future,FutureOption,Crypto,CryptoFuture,Cfd,IndexOption]

Potential Solution

Add

if (!security.IsTradable) continue;

to GetPortfolioTargets

Reproducing the Problem

Add [TestCase(SecurityType.Index, "SPX", 68)] to SignalExportManagerGetsCorrectPortfolioTargetArray.

Checklist

  • I have completely filled out this template
  • I have confirmed that this issue exists on the current master branch
  • I have confirmed that this is not a duplicate issue by searching issues
  • I have provided detailed steps to reproduce the issue
@AlexCatarino AlexCatarino changed the title Signal Export Generates Portofolio Targets for Non-Tradable Securities Signal Export Generates Portfolio Targets for Non-Tradable Securities Dec 5, 2024
@AlexCatarino AlexCatarino reopened this Dec 13, 2024
@AlexCatarino
Copy link
Member Author

Reopen since it has been observed with LEAN v2.5.0.0.16780:

Captura de ecrã 2024-12-13 172108

@AlexCatarino
Copy link
Member Author

AlexCatarino commented Dec 13, 2024

Code to reproduce the issue: c7ebc1
Note : It is not reproducible in backtest.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants