[fix]: marker strategy rename to remote

Author: pkarw

README.md

@@ -197,17 +197,19 @@ LLama 3.2 Vision Strategy is licensed on [Meta Community License Agreement](http
 Enabled by default. Please do use the `strategy=llama_vision` CLI and URL parameters to use it. It's by the way the default strategy
 
 
-### `marker`
+### `remote`
 
-[Marker, state of the art PDF OCR](https://github.com/VikParuchuri/marker) - works really great for more than 50 languages, including great accuracy for Polish and other languages - let's say that are "diffult" to read for standard OCR.
+Some OCR's - like [Marker, state of the art PDF OCR](https://github.com/VikParuchuri/marker) - works really great for more than 50 languages, including great accuracy for Polish and other languages - let's say that are "diffult" to read for standard OCR.
 
 The `marker-pdf` is however licensed on GPL3 license and **therefore it's not included** by default in this application (as we're bound to MIT). 
 
 The weights for the models are licensed cc-by-nc-sa-4.0, but I will waive that for any organization under $5M USD in gross revenue in the most recent 12-month period AND under $5M in lifetime VC/angel funding raised. You also must not be competitive with the Datalab API. If you want to remove the GPL license requirements (dual-license) and/or use the weights commercially over the revenue limit, check out the options here.
 
-To have it up and running please execute the following steps:
+To have it up and running you can execute the following steps:
 
 ```bash
+mkdir marker-distribution # this should be outside of the `text-extract-api` folder!
+cd marker-distribution
 pip install marker-pdf
 pip install -U uvicorn fastapi python-multipart
 marker_server --port 8002
@@ -216,16 +218,16 @@ marker_server --port 8002
 **Note: *** you might run `marker_server` on different port - then just make sure you export a proper env setting beffore starting off `text-extract-api` server:
 
 ```bash
-export MARKER_API_URL=http://localhost:8002/marker/upload
+export REMOTE_API_URL=http://localhost:8002/marker/upload
 ```
 
-Please do use the `strategy=marker` CLI and URL parameters to use it. For example:
+Please do use the `strategy=remote` CLI and URL parameters to use it. For example:
 
 ```bash
-curl -X POST -H "Content-Type: multipart/form-data" -F "file=@examples/example-mri.pdf" -F "strategy=marker" -F "ocr_cache=true" -F "prompt=" -F "model=" "http://localhost:8000/ocr/upload" 
+curl -X POST -H "Content-Type: multipart/form-data" -F "file=@examples/example-mri.pdf" -F "strategy=remote" -F "ocr_cache=true" -F "prompt=" -F "model=" "http://localhost:8000/ocr/upload" 
 ```
 
-We are connecting to marker via it's API to not share the same license (GPL3) by having it all linked on the source code level.
+We are connecting to remote OCR via it's API to not share the same license (GPL3) by having it all linked on the source code level.
 
 ## Getting started with Docker
 

config/strategies.yaml

@@ -5,5 +5,5 @@ strategies:
       class: text_extract_api.extract.strategies.minicpm_v.MiniCPMVStrategy
    easyocr:
       class: text_extract_api.extract.strategies.easyocr.EasyOCRStrategy
-   marker:
-      class: text_extract_api.extract.strategies.marker.MarkerStrategy
+   remote:
+      class: text_extract_api.extract.strategies.marker.RemoteStrategy

text_extract_api/extract/strategies/marker.py

@@ -2,21 +2,23 @@
 import tempfile
 import time
 
+from extract.extract_result import ExtractResult
+
 from text_extract_api.extract.strategies.strategy import Strategy
 from text_extract_api.files.file_formats.file_format import FileFormat
 from text_extract_api.files.file_formats.image import ImageFileFormat
 from text_extract_api.files.file_formats.pdf import PdfFileFormat
 import requests
 
 
-class MarkerStrategy(Strategy):
-    """Marker PDF via API - strategy"""
+class RemoteStrategy(Strategy):
+    """Remote API Strategy"""
 
     @classmethod
     def name(cls) -> str:
         return "marker"
 
-    def extract_text(self, file_format: FileFormat, language: str = 'en') -> str:
+    def extract_text(self, file_format: FileFormat, language: str = 'en') -> ExtractResult:
 
         if (
                 not isinstance(file_format, PdfFileFormat)
@@ -38,7 +40,7 @@ def extract_text(self, file_format: FileFormat, language: str = 'en') -> str:
             raise ValueError("No PDF file found - conversion error.")
 
         try: 
-            url = os.getenv("MARKER_API_URL", "http://localhost:8002/marker/upload")
+            url = os.getenv("REMOTE_API_URL", "http://localhost:8002/marker/upload")
             files = {'file': ('document.pdf', pdf_files[0].binary, 'application/pdf')}
             data = {
                 'page_range': None,
@@ -64,4 +66,4 @@ def extract_text(self, file_format: FileFormat, language: str = 'en') -> str:
             print('Error:', e)
             raise Exception("Failed to generate text with Marker PDF API. Make sure marker-pdf server is up and running: marker_server --port 8002. Details: https://github.com/VikParuchuri/marker")
 
-        return extracted_text
+        return ExtractResult.from_text(extracted_text)