Skip to content

Commit 6d94c25

Browse files
Pigbibicursoragent
andauthored
refactor: remove market_signal shim, bump QPK/UES pins (#211)
* refactor: remove market_signal shim, bump QPK/UES pins Co-authored-by: Cursor <cursoragent@cursor.com> * fix: align crypto/hk-equity strategy pins for QPK aee8121 Co-authored-by: Cursor <cursoragent@cursor.com> --------- Co-authored-by: Cursor <cursoragent@cursor.com>
1 parent 41650fd commit 6d94c25

4 files changed

Lines changed: 17 additions & 37 deletions

File tree

market_signal_runtime.py

Lines changed: 0 additions & 23 deletions
This file was deleted.

requirements.txt

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
flask
22
gunicorn
3-
quant-platform-kit @ git+https://github.com/QuantStrategyLab/QuantPlatformKit.git@b821e8c318e15d40f925c84a007ae335a3415cd5
4-
us-equity-strategies @ git+https://github.com/QuantStrategyLab/UsEquityStrategies.git@7ae083fae5c00c3df0c8fb1d98b045401a3b5bfa
5-
hk-equity-strategies @ git+https://github.com/QuantStrategyLab/HkEquityStrategies.git@e522688056229a0625031b2b14f34f360f3de548
3+
quant-platform-kit @ git+https://github.com/QuantStrategyLab/QuantPlatformKit.git@aee8121d530c2e92c72b68aee434bf174b3b9c85
4+
us-equity-strategies @ git+https://github.com/QuantStrategyLab/UsEquityStrategies.git@b2fa659304c02cc19f7c82e86b0ce36ef592846a
5+
hk-equity-strategies @ git+https://github.com/QuantStrategyLab/HkEquityStrategies.git@69a2e89e5c04fd495659909029db115492c4c940
66
pandas
77
requests
88
pytz

strategy_runtime.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@
1818
build_strategy_context_from_available_inputs,
1919
)
2020
from runtime_config_support import PlatformRuntimeSettings
21-
from market_signal_runtime import resolve_external_market_signal_inputs
21+
from us_equity_strategies.signals import resolve_external_market_signal_inputs
2222

2323
from strategy_loader import (
2424
load_strategy_entrypoint_for_profile,

tests/test_market_signal_runtime.py

Lines changed: 13 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -5,15 +5,18 @@
55

66
import pytest
77

8-
import market_signal_runtime
98
from us_equity_strategies.signals import runtime_market_signal_inputs as runtime_signal_inputs
109

1110

11+
def _resolve(*args, **kwargs):
12+
return runtime_signal_inputs.resolve_external_market_signal_inputs(*args, **kwargs)
13+
14+
1215
def test_unsupported_profile_does_not_load_market_signal():
1316
settings = SimpleNamespace(market_signal_required=True)
1417

1518
assert (
16-
market_signal_runtime.resolve_external_market_signal_inputs(
19+
runtime_signal_inputs.resolve_external_market_signal_inputs(
1720
strategy_profile="tqqq_growth_income",
1821
available_inputs={"derived_indicators"},
1922
runtime_settings=settings,
@@ -25,7 +28,7 @@ def test_unsupported_profile_does_not_load_market_signal():
2528
def test_ibit_without_reference_provides_empty_indicator_input():
2629
settings = SimpleNamespace(market_signal_required=False)
2730

28-
assert market_signal_runtime.resolve_external_market_signal_inputs(
31+
assert runtime_signal_inputs.resolve_external_market_signal_inputs(
2932
strategy_profile="ibit_smart_dca",
3033
available_inputs={"derived_indicators"},
3134
runtime_settings=settings,
@@ -36,7 +39,7 @@ def test_ibit_required_reference_missing_raises():
3639
settings = SimpleNamespace(market_signal_required=True)
3740

3841
with pytest.raises(RuntimeError, match="external market signal is required"):
39-
market_signal_runtime.resolve_external_market_signal_inputs(
42+
runtime_signal_inputs.resolve_external_market_signal_inputs(
4043
strategy_profile="ibit_smart_dca",
4144
available_inputs={"derived_indicators"},
4245
runtime_settings=settings,
@@ -46,7 +49,7 @@ def test_ibit_required_reference_missing_raises():
4649
def test_soxl_without_reference_preserves_legacy_inputs():
4750
settings = SimpleNamespace(market_signal_required=False)
4851

49-
assert market_signal_runtime.resolve_external_market_signal_inputs(
52+
assert runtime_signal_inputs.resolve_external_market_signal_inputs(
5053
strategy_profile="soxl_soxx_trend_income",
5154
available_inputs={"derived_indicators"},
5255
runtime_settings=settings,
@@ -56,7 +59,7 @@ def test_soxl_without_reference_preserves_legacy_inputs():
5659
def test_nasdaq_without_reference_preserves_legacy_inputs():
5760
settings = SimpleNamespace(market_signal_required=False)
5861

59-
assert market_signal_runtime.resolve_external_market_signal_inputs(
62+
assert runtime_signal_inputs.resolve_external_market_signal_inputs(
6063
strategy_profile="nasdaq_sp500_smart_dca",
6164
available_inputs={"derived_indicators"},
6265
runtime_settings=settings,
@@ -70,7 +73,7 @@ def test_soxl_required_reference_missing_raises():
7073
RuntimeError,
7174
match="soxl_soxx_trend_income external market signal is required",
7275
):
73-
market_signal_runtime.resolve_external_market_signal_inputs(
76+
runtime_signal_inputs.resolve_external_market_signal_inputs(
7477
strategy_profile="soxl_soxx_trend_income",
7578
available_inputs={"derived_indicators"},
7679
runtime_settings=settings,
@@ -120,7 +123,7 @@ def fake_extract(
120123
market_signal_max_stale_days=5,
121124
)
122125

123-
assert market_signal_runtime.resolve_external_market_signal_inputs(
126+
assert runtime_signal_inputs.resolve_external_market_signal_inputs(
124127
strategy_profile="ibit_smart_dca",
125128
available_inputs={"derived_indicators"},
126129
runtime_settings=settings,
@@ -188,7 +191,7 @@ def fake_extract(
188191
market_signal_max_stale_days=4,
189192
)
190193

191-
assert market_signal_runtime.resolve_external_market_signal_inputs(
194+
assert runtime_signal_inputs.resolve_external_market_signal_inputs(
192195
strategy_profile="nasdaq_sp500_smart_dca",
193196
available_inputs={"derived_indicators"},
194197
runtime_settings=settings,
@@ -255,7 +258,7 @@ def fake_extract(
255258
market_signal_fallback_mode="none",
256259
)
257260

258-
assert market_signal_runtime.resolve_external_market_signal_inputs(
261+
assert runtime_signal_inputs.resolve_external_market_signal_inputs(
259262
strategy_profile="soxl_soxx_trend_income",
260263
available_inputs={"derived_indicators"},
261264
runtime_settings=settings,

0 commit comments

Comments
 (0)