Skip to content

Add waymark-jsonlines#289

Merged
MOZGIII merged 1 commit intomainfrom
mzg/2026-03-26/jsonlines
Apr 12, 2026
Merged

Add waymark-jsonlines#289
MOZGIII merged 1 commit intomainfrom
mzg/2026-03-26/jsonlines

Conversation

@MOZGIII
Copy link
Copy Markdown
Collaborator

@MOZGIII MOZGIII commented Mar 26, 2026

This PR adds machinery for reading and writing serde-capable data items as JSONLines format.

I'm considering JSONLines format for replay feature logs - so the primary use-case is expected to be to stream action completions log.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 26, 2026

Coverage Report

Python Coverage

Metric Coverage
Lines 72.0%
Branches 58.0%

Download HTML Report

Rust Coverage

Metric Coverage
Lines 66.2% 🟢 (+0.2%)
Branches N/A

Download HTML Report

Compared to main branch

Comment thread crates/lib/jsonlines/src/lib.rs Outdated
@MOZGIII MOZGIII marked this pull request as draft March 26, 2026 18:42
@MOZGIII
Copy link
Copy Markdown
Collaborator Author

MOZGIII commented Mar 27, 2026

Btw, due to high volume of data I'm going to investigate more compact formats, like bson and parquet & arrow

@MOZGIII MOZGIII force-pushed the mzg/2026-03-26/jsonlines branch 2 times, most recently from 00e9206 to e94fb30 Compare April 10, 2026 12:07
@MOZGIII MOZGIII marked this pull request as ready for review April 10, 2026 12:07
@MOZGIII MOZGIII requested a review from piercefreeman April 10, 2026 12:07
@MOZGIII
Copy link
Copy Markdown
Collaborator Author

MOZGIII commented Apr 10, 2026

@piercefreeman I've reworked this crate, it is now much more idiomatic and less bloated

@MOZGIII MOZGIII force-pushed the mzg/2026-03-26/jsonlines branch from e94fb30 to 70a2647 Compare April 10, 2026 19:05
@MOZGIII MOZGIII merged commit c1e6fd5 into main Apr 12, 2026
15 checks passed
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.

2 participants