]> Piment Noir Git Repositories - freqai-strategies.git/commit
refactor(quickadapter): share Optuna-artifact quarantine path helper
authorJérôme Benoit <jerome.benoit@piment-noir.org>
Tue, 4 Aug 2026 21:37:14 +0000 (23:37 +0200)
committerJérôme Benoit <jerome.benoit@piment-noir.org>
Tue, 4 Aug 2026 21:37:14 +0000 (23:37 +0200)
commit5bdf42c54241453a9fdcb6bec4734d43f3a27e56
tree6ca908345f3d2892a196d4bdb9b350a959702322
parentbfd663bfab8e2ef35129fb87b557c12a97bc1f5c
refactor(quickadapter): share Optuna-artifact quarantine path helper

The best-params quarantine in Utils re-inlined the timestamp + tie-break
path computation that the journal quarantine already implemented in
QuickAdapterRegressorV3, duplicating the algorithm across the two
modules.

- Add a shared, parameterized _optuna_quarantine_path(path, now, *, tag,
  limit) in Utils (the module both call sites can import).
- Delegate from _quarantine_corrupt_optuna_best_params (Utils) and from
  _optuna_quarantine_journal (regressor); each keeps its own domain
  constants (journal vs best-params), logging and rename handling, so
  quarantine names, log output and error semantics are unchanged.
- Remove the regressor's private _optuna_quarantine_path copy.
quickadapter/user_data/freqaimodels/QuickAdapterRegressorV3.py
quickadapter/user_data/strategies/Utils.py