Skip to content

Pin UsEquityStrategies to 4e638c5 #116

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.

Sign up for GitHub

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

Merged
Pigbibi merged 1 commit into from
Jun 19, 2026
Merged

Pin UsEquityStrategies to 4e638c5 #116

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion pyproject.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ authors = [
dependencies = [
"firstrade==0.0.39",
"quant-platform-kit @ git+https://github.com/QuantStrategyLab/QuantPlatformKit.git@46ca4ea3de8f98a58e2dd86158e7f2070d085cd1",
"us-equity-strategies @ git+https://github.com/QuantStrategyLab/UsEquityStrategies.git@31406abfb15507270992e62ead8d1068c03328d0",
"us-equity-strategies @ git+https://github.com/QuantStrategyLab/UsEquityStrategies.git@4e638c5fc880253bf22e2e056630fbce6588256a",

@chatgpt-codex-connector chatgpt-codex-connector Bot Jun 19, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Wire Nasdaq DCA signal loading before pinning

With this pin, UsEquityStrategies adds unified derived-indicator consumption for nasdaq_sp500_smart_dca, but the platform resolver still returns {} for every profile except ibit_smart_dca (market_signal_runtime.py:29-31). In a Nasdaq Smart DCA deployment configured with FIRSTRADE_MARKET_SIGNAL_*/MARKET_SIGNAL_REQUIRED, the handoff artifacts are silently skipped and required signals are not enforced, so the strategy falls back to broker history instead of the unified technical signal bundle this version is meant to consume; add the Nasdaq consumer mapping before advancing the dependency.

Useful? React with 👍 / 👎.

"google-cloud-storage",
"requests",
]
Expand Down
2 changes: 1 addition & 1 deletion requirements.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ flask
gunicorn
firstrade==0.0.39
quant-platform-kit @ git+https://github.com/QuantStrategyLab/QuantPlatformKit.git@46ca4ea3de8f98a58e2dd86158e7f2070d085cd1
us-equity-strategies @ git+https://github.com/QuantStrategyLab/UsEquityStrategies.git@31406abfb15507270992e62ead8d1068c03328d0
us-equity-strategies @ git+https://github.com/QuantStrategyLab/UsEquityStrategies.git@4e638c5fc880253bf22e2e056630fbce6588256a
google-cloud-storage
google-auth
requests
Expand Down