mirror of
https://github.com/microsoft/qlib.git
synced 2026-11-13 22:41:11 +08:00
* security: harden artifact serialization and loading * ci: restore dependency and title lint compatibility * fix: address artifact test and lint failures * ci: handle current MLflow and Windows OSQP changes * ci: use lxml release with cross-platform wheels * fix: defer optional CVXPY solver imports * docs: mark notebook model artifacts as trusted * docs: mark position artifacts as trusted * style: document intentional lazy import * ci: skip broken released OSQP on Windows Python 3.8 * fix: mark online model artifacts as trusted * test: make MLflow client timing check robust * ci: retry nondeterministic benchmark smoke test * fix: cap filelock before fork safety regression * fix: use filelock release compatible with multiprocessing * fix: preserve artifact loading compatibility after hardening * fix(security): validate derived artifact paths and extend regressions * fix(security): preserve extended data artifact round trips * fix(security): retain legacy NumPy and pandas artifact classes Allow the exact MaskedArray and SparseDtype module paths emitted by the older dependency versions in CI. Preserve restricted loading and cover both old and new paths. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * fix(data): avoid unused trading calendar lookup in PIT normalization PIT normalization uses report dates and calendar-day offsets, so do not request an unused Baostock calendar. Cover annual and quarterly normalization without online calendar access. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * fix(security): require explicit trust in artifact workflows Expose default-off artifact trust at online, delayed-training and meta-workflow entry points. Keep data loading restricted, preserve legacy default recorder calls and restored workflow defaults, and document migration and trust boundaries. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * test(workflow): cover complete DDG-DA and HIST artifact flows Exercise real training, cache and recorder restoration, saved-task replay, signals and portfolio backtests. Fix the cache trust, grouping, relative-path, LightGBM early-stopping and HIST CPU blockers exposed by full execution. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * fix(security)!: unify artifact trust and publish upgrade guidance Use trusted consistently across public APIs and CLI options. Migrate pre-release saved consent only at serialization boundaries, reject invalid or conflicting settings, and preserve restricted data reads. Add a canonical upgrade guide with README, installation, changelog and runtime-error entry points. BREAKING CHANGE: executable artifact restoration requires explicit trusted=True. HIST mappings use JSON and high-frequency cache paths must remain within artifact_root. See docs/start/artifact_migration.rst for upgrading existing workflows. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * fix(security): remove trial-only trust compatibility Target upgrades from pre-PR Qlib rather than artifacts generated by intermediate PR revisions. Retain restricted defaults for saved objects without consent, unify current APIs, and remove state/keyword migration branches and documentation. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * ci: avoid failed dataset probes and space download retries Download the same existing v2 archives used by successful jobs without probing nonexistent package-version assets. Allow five download attempts 60 seconds apart; preserve the full matrix, datasets, noninteractive setup and failure propagation. Cover the exact archives and retry policy in offline CI tests. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * docs: add practical artifact migration examples Show explicit consent for restored online components while preserving the original manager, schema-aware Arrow conversions, timezone-preserving ZoneInfo migration, and restricted round trips into a separate recorder. Explain shared-component scope, staging artifacts and trusted-source prerequisites. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * fix(workflow): preserve fresh artifact workflows with scoped trust Keep immutable cache references intact at open rolling tails. Add explicit per-load record-template consent and per-artifact collector loading options without widening other reads. Restore the original template class after parent-load failures, and support audited business-frequency and NumPy record representations through exact pickle globals. Document the scoped APIs and cover fresh trained-model reads, cache generation, numerical round trips and refusal paths. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
148 lines
5.6 KiB
Python
148 lines
5.6 KiB
Python
import inspect
|
|
import pickle
|
|
import runpy
|
|
import warnings
|
|
from importlib import import_module
|
|
from pathlib import Path
|
|
|
|
import numpy as np
|
|
import pytest
|
|
from fire.helptext import HelpText
|
|
|
|
from qlib.utils.pickle_utils import ARTIFACT_MIGRATION_URL
|
|
|
|
|
|
WORKFLOWS = [
|
|
("qlib.workflow.online.strategy", "RollingStrategy"),
|
|
("qlib.workflow.online.utils", "OnlineToolR"),
|
|
("qlib.workflow.online.update", "RMDLoader"),
|
|
("qlib.workflow.online.update", "PredUpdater"),
|
|
("qlib.workflow.online.update", "LabelUpdater"),
|
|
("qlib.model.trainer", "DelayTrainerR"),
|
|
("qlib.model.trainer", "DelayTrainerRM"),
|
|
("qlib.contrib.rolling.ddgda", "DDGDA"),
|
|
]
|
|
|
|
|
|
@pytest.fixture(params=WORKFLOWS, ids=[name for _, name in WORKFLOWS])
|
|
def workflow_cls(request):
|
|
module, name = request.param
|
|
return getattr(import_module(module), name)
|
|
|
|
|
|
@pytest.mark.parametrize("consent", [False, True])
|
|
def test_saved_consent_survives_restoring_a_workflow(workflow_cls, consent):
|
|
original = object.__new__(workflow_cls)
|
|
original.trusted = consent
|
|
original.marker = "preserved"
|
|
with warnings.catch_warnings(record=True) as caught:
|
|
warnings.simplefilter("always")
|
|
restored = pickle.loads(pickle.dumps(original))
|
|
assert not caught
|
|
assert restored.trusted is consent
|
|
assert restored.__dict__ == {"trusted": consent, "marker": "preserved"}
|
|
|
|
|
|
def test_saved_workflow_without_consent_does_not_gain_it(workflow_cls):
|
|
original = object.__new__(workflow_cls)
|
|
with warnings.catch_warnings(record=True) as caught:
|
|
warnings.simplefilter("always")
|
|
restored = pickle.loads(pickle.dumps(original))
|
|
assert not caught
|
|
assert restored.trusted is False
|
|
assert "trusted" not in restored.__dict__
|
|
|
|
|
|
def test_old_online_manager_requires_consent_on_each_component():
|
|
from qlib.model.trainer import DelayTrainerR
|
|
from qlib.workflow.online.manager import OnlineManager
|
|
from qlib.workflow.online.strategy import RollingStrategy
|
|
from qlib.workflow.online.utils import OnlineToolR
|
|
|
|
manager = object.__new__(OnlineManager)
|
|
strategy = object.__new__(RollingStrategy)
|
|
strategy.tool = object.__new__(OnlineToolR)
|
|
manager.strategies = [strategy]
|
|
manager.trainer = object.__new__(DelayTrainerR)
|
|
|
|
with warnings.catch_warnings(record=True) as caught:
|
|
warnings.simplefilter("always")
|
|
restored = pickle.loads(pickle.dumps(manager))
|
|
|
|
assert not caught
|
|
assert restored.strategies[0].trusted is False
|
|
assert restored.strategies[0].tool.trusted is False
|
|
assert restored.trainer.trusted is False
|
|
|
|
restored.strategies[0].trusted = True
|
|
reloaded = pickle.loads(pickle.dumps(restored))
|
|
assert reloaded.strategies[0].trusted is True
|
|
assert reloaded.strategies[0].tool.trusted is False
|
|
assert reloaded.trainer.trusted is False
|
|
|
|
|
|
def test_public_workflow_constructors_default_to_restricted_loading(workflow_cls):
|
|
parameters = inspect.signature(workflow_cls).parameters
|
|
assert parameters["trusted"].default is False
|
|
|
|
|
|
def _entry_points():
|
|
from qlib.contrib.meta.data_selection.dataset import InternalData, MetaDatasetDS
|
|
from qlib.contrib.rolling.ddgda import DDGDA
|
|
from qlib.model.trainer import DelayTrainerR, DelayTrainerRM, end_task_train
|
|
from qlib.workflow import QlibRecorder
|
|
from qlib.workflow.online.strategy import RollingStrategy
|
|
from qlib.workflow.online.update import LabelUpdater, PredUpdater, RMDLoader
|
|
from qlib.workflow.online.utils import OnlineToolR
|
|
from qlib.workflow.recorder import MLflowRecorder, Recorder
|
|
from qlib.workflow.record_temp import RecordTemp
|
|
|
|
return [
|
|
(RMDLoader, {"rec": None}),
|
|
(PredUpdater, {"record": None}),
|
|
(LabelUpdater, {"record": None}),
|
|
(OnlineToolR, {}),
|
|
(RollingStrategy, {"name_id": "unused", "task_template": {}, "rolling_gen": None}),
|
|
(DelayTrainerR, {}),
|
|
(DelayTrainerRM, {}),
|
|
(DDGDA, {"conf_path": "unused.yaml"}),
|
|
(MetaDatasetDS, {"task_tpl": [], "step": 20, "exp_name": "unused", "segments": 0.5}),
|
|
(InternalData({}, 20, "unused").setup, {}),
|
|
(end_task_train, {"rec": None, "experiment_name": "unused"}),
|
|
(QlibRecorder(None).load_object, {"name": "unused"}),
|
|
(RecordTemp(None).load, {"name": "unused"}),
|
|
(MLflowRecorder.load_object, {}),
|
|
(Recorder.load_object, {}),
|
|
]
|
|
|
|
|
|
def test_all_entry_points_use_the_same_public_keyword():
|
|
for entry, _ in _entry_points():
|
|
parameters = inspect.signature(entry).parameters
|
|
assert parameters["trusted"].default is False, entry
|
|
|
|
|
|
@pytest.mark.parametrize("value", ["false", 0, 1, None, np.bool_(True)])
|
|
def test_workflow_entries_reject_non_boolean_consent_before_loading(value):
|
|
for entry, kwargs in _entry_points()[:-2]:
|
|
with pytest.raises(TypeError, match="`trusted` must be a bool") as caught:
|
|
entry(**kwargs, trusted=value)
|
|
assert ARTIFACT_MIGRATION_URL in str(caught.value)
|
|
|
|
|
|
@pytest.mark.parametrize(
|
|
"path,class_name",
|
|
[
|
|
("examples/online_srv/update_online_pred.py", "UpdatePredExample"),
|
|
("examples/online_srv/online_management_simulate.py", "OnlineSimulationExample"),
|
|
("examples/online_srv/rolling_online_management.py", "RollingOnlineExample"),
|
|
("examples/benchmarks_dynamic/DDG-DA/workflow.py", "DDGDABench"),
|
|
],
|
|
)
|
|
def test_example_cli_help_exposes_the_same_trust_flag(path, class_name):
|
|
namespace = runpy.run_path(str(Path(__file__).resolve().parents[2] / path))
|
|
example = namespace[class_name]
|
|
assert inspect.signature(example).parameters["trusted"].default is False
|
|
text = HelpText(example)
|
|
assert "--trusted" in text
|