• Stefy Lanza (nextime / spora )'s avatar
    ocr: olmOCR has to trust a pod's certificate itself, and U+FFFD is one token... · dc86733f
    Stefy Lanza (nextime / spora ) authored
    ocr: olmOCR has to trust a pod's certificate itself, and U+FFFD is one token not two bytes (v0.3.22)
    
    olmOCR could not reach a rented pod at all: CERTIFICATE_VERIFY_FAILED. A pod on
    the direct-TCP path serves HTTPS with a certificate signed by THIS install's own
    CA (codai/api/pod_tls.py), at a public IP that was not known when the certificate
    was issued, so stock `requests` fails it twice over.
    
    Surya never hit this because surya attaches through the fan-out gateway, which
    learned to trust pod certificates in 0.3.16. olmOCR talks to the upstream the
    dispatcher chose DIRECTLY, so it needed the same trust of its own -- and the gap
    only surfaced when olmOCR was asked to cover for surya, which is exactly when a
    fallback engine must not be the thing that is broken. It now goes through
    codai/api/pod_http.py, which verifies https-to-a-bare-IP against our CA with the
    hostname check off and hands every other URL -- localhost, a named server,
    another coderai -- to requests untouched. Swept the rest of the tree: every other
    raw requests call targets a localhost worker or RunPod's own public API, where
    pod_http would be a pass-through anyway.
    
    test_ocr_olmocr's HTTP-status test patched sys.modules["requests"] wholesale,
    which now breaks pod_http's own import of it; it patches pod_http.post instead
    and asserts the same two statuses.
    
    Also records what the accent investigation actually established, because the
    natural reading of U+FFFD is wrong and expensive. It is NOT a charset bug:
    U+FFFD is a real entry in surya-ocr-2's vocabulary (id 65421 of 65425, the only
    id that decodes to it), the tokenizer is WordLevel and character-level for Latin
    so an accent cannot be split across tokens, and the decisive detail is the COUNT
    -- a byte-level mis-decode yields TWO markers per accented letter (a-grave is two
    UTF-8 bytes, each one undecodable), and what arrives is ONE. One marker is one
    token. Every other hop was checked: _html_to_text is regex plus html.unescape and
    survives literal and entity forms; the isolated-venv worker emits json.dumps with
    ensure_ascii=True, so the pipe carries pure-ASCII escapes and the locale cannot
    matter; the gateway forwards raw bytes.
    
    And it corrects the previous commit's claim. Raising DPI is right on its own
    merits -- 200 really was feeding surya 62% of the pixels it accepts -- but a
    production document came back with the same markers at 200 AND at 300, which is
    evidence against too-few-pixels being the whole story: a perception failure would
    be stochastic, not every accent every time. The docs now say what distinguishes
    the remaining candidates (the document's embedded scan resolution, which
    `pdfimages -list` reports and which no rasterisation DPI can improve, versus the
    serving stack) and how to tell them apart: OCR perfect synthetic Italian through
    the same path.
    Co-Authored-By: 's avatarClaude Opus 5 <noreply@anthropic.com>
    Claude-Session: https://claude.ai/code/session_01PxYHWbCAjjjD7A1rhFKtaW
    dc86733f
Name
Last commit
Last update
codai Loading commit data...
docs Loading commit data...
packaging Loading commit data...
samples Loading commit data...
tests Loading commit data...
third_party Loading commit data...
tools Loading commit data...
.dockerignore Loading commit data...
.gitignore Loading commit data...
AI.PROMPT Loading commit data...
CODERAI_API_DOCUMENTATION.md Loading commit data...
CoderAI.gif Loading commit data...
DISTRIBUTION.md Loading commit data...
LICENSE.md Loading commit data...
MULTIMODAL_CAPABILITIES.md Loading commit data...
MULTIMODAL_UI_EXAMPLES.md Loading commit data...
README.md Loading commit data...
REQUEST-geoclip-embeddings.md Loading commit data...
REQUEST-vpr-embeddings.md Loading commit data...
amdgpu-coredump-05000.bin Loading commit data...
build-oci.sh Loading commit data...
build.ps1 Loading commit data...
build.sh Loading commit data...
coderai Loading commit data...
coderai-broker-implementation-reference.md Loading commit data...
coderai-integration.md Loading commit data...
commands Loading commit data...
osxbuild.sh Loading commit data...
package-oci.sh Loading commit data...
package-tarball.sh Loading commit data...
requirements-crisperwhisper.txt Loading commit data...
requirements-h3.txt Loading commit data...
requirements-longcat-train.txt Loading commit data...
requirements-longcat.txt Loading commit data...
requirements-melotts.txt Loading commit data...
requirements-nemo.txt Loading commit data...
requirements-nvidia.txt Loading commit data...
requirements-ocr-paddle.txt Loading commit data...
requirements-ocr.txt Loading commit data...
requirements-pyannote.txt Loading commit data...
requirements-surya.txt Loading commit data...
requirements-vllm.txt Loading commit data...
requirements-vulkan.txt Loading commit data...
requirements.txt Loading commit data...
run-oci.sh Loading commit data...
smoke-test-oci.sh Loading commit data...
todo.md Loading commit data...