Skip to content

Type Model Target mock JSON - #3590

Open
adamtheturtle wants to merge 5 commits into
mainfrom
fix/model-target-json-types
Open

Type Model Target mock JSON#3590
adamtheturtle wants to merge 5 commits into
mainfrom
fix/model-target-json-types

Conversation

@adamtheturtle

@adamtheturtle adamtheturtle commented Sep 9, 2026

Copy link
Copy Markdown
Member

Summary

  • define and document a recursive JSON value type with the public Model Target types
  • use it for persisted dataset requests, generation warnings, status response bodies, and Model Target Web API validation
  • remove seven Pyrefly suppressions without replacing them with broader ignores

Validation

  • uv run --group=dev prek run --all-files --hook-stage pre-commit
  • uv run --group=dev prek run --all-files --hook-stage pre-push
  • documentation and spelling builds
  • focused public Model Target and Flask integration paths (573 passed, 244 live-service cases skipped)

The complete local suite also includes live Vuforia cases and requires external credentials; CI covers that matrix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant