Align ReadingOrderModel with enabled pattern used by other models

Replace skip_prediction option in ReadingOrderOptions with an enabled
parameter in ReadingOrderModel.__init__, consistent with AutoOcrModel
and other pipeline models. The do_reading_order flag in
PdfPipelineOptions now controls this enabled argument directly.

Signed-off-by: stone <frank.schruefer@t-online.de>

Author: Frank-Schruefer

docling/models/stages/reading_order/readingorder_model.py

@@ -37,11 +37,11 @@ class ReadingOrderOptions(BaseModel):
     model_config = ConfigDict(protected_namespaces=())
 
     model_names: str = ""  # e.g. "language;term;reference"
-    skip_prediction: bool = False  # if True, keep layout postprocessor order as-is
 
 
 class ReadingOrderModel:
-    def __init__(self, options: ReadingOrderOptions):
+    def __init__(self, enabled: bool = True, options: ReadingOrderOptions = ReadingOrderOptions()):
+        self.enabled = enabled
         self.options = options
         self.ro_model = ReadingOrderPredictor()
         self.list_item_processor = ListItemMarkerProcessor()
@@ -434,7 +434,7 @@ def __call__(self, conv_res: ConversionResult) -> DoclingDocument:
             page_elements = self._assembled_to_readingorder_elements(conv_res)
 
             # Apply reading order
-            if self.options.skip_prediction:
+            if not self.enabled:
                 sorted_elements = page_elements
             else:
                 sorted_elements = self.ro_model.predict_reading_order(

docling/pipeline/standard_pdf_pipeline.py

@@ -503,9 +503,10 @@ def _init_models(self) -> None:
             enable_remote_services=self.pipeline_options.enable_remote_services,
         )
         self.assemble_model = PageAssembleModel(options=PageAssembleOptions())
-        self.reading_order_model = ReadingOrderModel(options=ReadingOrderOptions(
-            skip_prediction=not self.pipeline_options.do_reading_order
-        ))
+        self.reading_order_model = ReadingOrderModel(
+            enabled=self.pipeline_options.do_reading_order,
+            options=ReadingOrderOptions(),
+        )
 
         # --- optional enrichment ------------------------------------------------
         # Create a copy to avoid mutating pipeline_options in-place,