Skip to content

feat(mojo): add mojo-json client from leo-gan/gld-json - #142

Merged
leo-gan merged 1 commit into
masterfrom
add-mojo-json-client
Sep 9, 2026
Merged

feat(mojo): add mojo-json client from leo-gan/gld-json#142
leo-gan merged 1 commit into
masterfrom
add-mojo-json-client

Conversation

@leo-gan

@leo-gan leo-gan commented Sep 9, 2026

Copy link
Copy Markdown
Owner

Summary

  • Add a mojo-json harness client for leo-gan/gld-json 0.2.0
  • Vendor sources as gldjson_* so they can share a pixi env with mojo-avro
  • Register the serializer in benchmark_config.yaml and every experiment that already lists EmberJson
  • Refresh Mojo dashboard + experiment results from full run 2026-09-09-131928

Validation

  • Tests: analysis 90 passed; python 32 passed; javascript 10 passed; rust 15 passed; pixi run test in mojo/ printed ok
  • Benchmarks: full, langs mojo, stem 2026-09-09-131928 (7000 rows, fidelity 1.0 for all 7 serializers including mojo-json)
  • Experiments: timed 14 folders for mojo
  • Error CSVs: clean (0 error rows)
  • Analysis: reports/mojo snapshot from that stem; docs/mojo/index.md lists the new row
  • Dashboard: sync-data.pymojo_latest.json.gz + stats_mojo_latest.json.gz + available_runs.json + experiment catalog (other language payloads left unchanged)

Notes

  • config/benchmark_config.yaml only adds a Mojo serializer. Full bench was scoped with PREPARE_PR_LANGS=mojo rather than re-running every language.
  • The client uses vendored WireWriter / WireReader expected-order encode/decode of the suite types. Package names are rewritten because mojo-avro already owns json / runtime / wire.

Vendor gld-json as gldjson_* so it can share a pixi env with mojo-avro.
Wire expected-order encode/decode into the runner and roundtrip tests,
register the row in config and experiments, and refresh Mojo dashboard
payloads from full run 2026-09-09-131928 (fidelity 1.0).
@leo-gan
leo-gan merged commit e7291d4 into master Sep 9, 2026
15 checks passed
@leo-gan
leo-gan deleted the add-mojo-json-client branch September 9, 2026 20:29
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