From 3aa610f0f7af30807680b5fbe8c1e991e7202cb7 Mon Sep 17 00:00:00 2001 From: =?utf8?q?J=C3=A9r=C3=B4me=20Benoit?= Date: Tue, 4 Aug 2026 23:52:08 +0200 Subject: [PATCH] docs(reforcexy): document quarantine-path glob-avoidance rationale Mirror the rationale already documented on the shared QuickAdapter helper: _quarantine_path appends the tag and timestamp after the complete filename (extension included) so live-artefact globs never match quarantined files. Docstring only; no behavior change. --- ReforceXY/user_data/freqaimodels/ReforceXY.py | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/ReforceXY/user_data/freqaimodels/ReforceXY.py b/ReforceXY/user_data/freqaimodels/ReforceXY.py index 9572e18..2a8f0b0 100644 --- a/ReforceXY/user_data/freqaimodels/ReforceXY.py +++ b/ReforceXY/user_data/freqaimodels/ReforceXY.py @@ -1443,6 +1443,13 @@ class ReforceXY(BaseReinforcementLearningModel): @staticmethod def _quarantine_path(path: Path, now: datetime) -> Path: + """Quarantine target path for a corrupt Optuna artefact. + + The tag and timestamp are appended after the complete filename + (extension included) so live-artefact globs never match quarantined + files. Collisions are bounded by ``_QUARANTINE_TIE_BREAK_LIMIT``; + exhausted candidates raise instead of reusing a quarantine file. + """ stamp = now.strftime("%Y%m%dT%H%M%S%fZ") base_name = f"{path.name}.{ReforceXY._QUARANTINE_TAG}-{stamp}" for index in range(ReforceXY._QUARANTINE_TIE_BREAK_LIMIT + 1): -- 2.53.0