From 308812cf7315fe896dbcd319493277d82aa36bd2 Mon Sep 17 00:00:00 2001 From: ClemensBuechner Date: Tue, 16 Jul 2024 14:24:45 +0200 Subject: [PATCH] [trivial] Clean up documentation of iterated search. --- .../search_algorithms/iterated_search.cc | 19 +++++++------------ 1 file changed, 7 insertions(+), 12 deletions(-) diff --git a/src/search/search_algorithms/iterated_search.cc b/src/search/search_algorithms/iterated_search.cc index 5755623be2..23c009a4d8 100644 --- a/src/search/search_algorithms/iterated_search.cc +++ b/src/search/search_algorithms/iterated_search.cc @@ -164,7 +164,8 @@ class IteratedSearchFeature "Note 1", "We don't cache heuristic values between search iterations at" " the moment. If you perform a LAMA-style iterative search," - " heuristic values will be computed multiple times."); + " heuristic values and other per-state information will be computed" + " multiple times."); document_note( "Note 2", "The configuration\n```\n" @@ -173,18 +174,12 @@ class IteratedSearchFeature "lazy_wastar([ipdb()],w=2), lazy_wastar([ipdb()],w=1)])\"\n" "```\nwould perform the preprocessing phase of the ipdb heuristic " "5 times (once before each iteration).\n\n" - "To avoid this, use heuristic predefinition, which avoids duplicate " - "preprocessing, as follows:\n```\n" - "--evaluator \"h=ipdb()\" --search " - "\"iterated([lazy_wastar([h],w=10), lazy_wastar([h],w=5), lazy_wastar([h],w=3), " - "lazy_wastar([h],w=2), lazy_wastar([h],w=1)])\"\n" + "To avoid this, use heuristic predefinition, which avoids " + "duplicate preprocessing, as follows:\n```\n" + "\"let(h,ipdb(),iterated([lazy_wastar([h],w=10), " + "lazy_wastar([h],w=5), lazy_wastar([h],w=3), lazy_wastar([h],w=2), " + "lazy_wastar([h],w=1)]))\"\n" "```"); - document_note( - "Note 3", - "If you reuse the same landmark count heuristic " - "(using heuristic predefinition) between iterations, " - "the path data (that is, landmark status for each visited state) " - "will be saved between iterations."); } virtual shared_ptr create_component(const plugins::Options &options, const utils::Context &context) const override {